Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Objectives
Use
find(id)
instead offind_by(id: id)
. It's preferable to raise a 404 HTML error than any other unexpected/unknown Exception.Add the methods
find_by_slug_or_id
andfind_by_slug_or_id!
to the moduleSluggable
to make it easier to add these methods other models.Does this PR need a Backport to CONSUL?
The following commits can be backported now:
The last commit 1b6c1ee, it would be nice to be able to
find_by_slug
also in CONSULBut, we will probably do it in a future PR, we will have to include at least this other PR #520