-
Notifications
You must be signed in to change notification settings - Fork 4
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
fix(templates): fix error handling and route method call #83
Conversation
|
🥷 Code experts: no user matched threshold 10 See details
Knowledge based on git-blame:
Knowledge based on git-blame:
Knowledge based on git-blame:
Knowledge based on git-blame:
Knowledge based on git-blame:
Knowledge based on git-blame: To learn more about /:\ gitStream - Visit our Docs |
application-templates/javascript/event/src/routes/event.route.js
Outdated
Show resolved
Hide resolved
application-templates/javascript/event/src/routes/event.route.js
Outdated
Show resolved
Hide resolved
6dd8d90
to
cfd445a
Compare
Link to the Jira issue
This PR will englobe two tickets
IM-1114 and IM-1117
Description and context
Since the work was fairly straightforward, we used just one PR to fix both tickets.
So, this PR will close #78 and close #80.
Assessing #78, a new implementation was introduced, as suggested, to call the next() function inside a
try-catch
block.Regarding #80, the following files were updated to call the correct controller methods inside the routes.
Type of change