-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
added Request form api #28
Conversation
…eeper into crudRequestForms
import org.springframework.web.bind.annotation.ResponseStatus; | ||
|
||
@ResponseStatus(HttpStatus.BAD_REQUEST) | ||
public class PDFNotConversionException extends Exception{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FailedFileConversionException?
If you want to keep the same name, pls add a javadoc mentioning what this exception means
throw new PDFNotConversionException("Could not store pdf"); | ||
} | ||
|
||
return requestFormService.save(requestForm); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This returns the entire thing as JSON. how is the file returned?
If as a json array, that will make the response too huge, could you check?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor things to take a look at.
Overall seems in good shape.
ignore this pr,made a newer one with slightly cleaner code |
Please close if not required. |
need to remove RequestDTO class, no use of it . update the put mapping in the next Pr
made an urgent pr !