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.
Where
What
How
Screenshots
Test
Notes
The model created on this PR takes the custom validation for
video_url
implemented in Allow only YouTube/Vimeo URLs on 'video_url' attribute #1854, which it's not DRY at all. @iagirre created a concern to solve Create a Video concern to abstractvideo_url
attributes #1865 and the PR is pending. As soon as that PR is done and merged, I'll handle the refactor to maximize DRYness.Some errors were made on Admin::Poll questions UI refactor #1977, but were fixed on this PR. See commits' description for further information.
Since Admin question answer documents #1988 is done and waiting for approval/merge, we should be careful when merging since both PRs handle the
routes.rb
file and English/Spanish locales, as well as the Questions 'show' view. This is important since we want to keep the markup clean and tidy