This repository has been archived by the owner on Jun 14, 2024. It is now read-only.
[TASK] Add questions update visible_options validation
#75
Assignees
Comments
|
Hey there @jfcalvo! 👋 Fancy seeing you here again. Diving into your issue now and I'll get back to you with a detailed response soon. 🕵️♂️ |
15 tasks
15 tasks
jfcalvo
added a commit
that referenced
this issue
Mar 19, 2024
# Description This PR add a new validation to `QuestionUpdateValidator` class so we are now validating `visible_options` attribute when it's updated. The validation will avoid that the number specified for `visible_options` is bigger than the number of available options defined by the question. Closes #75 **Type of change** (Please delete options that are not relevant. Remember to title the PR according to the type of change) - [ ] Bug fix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] Refactor (change restructuring the codebase without changing functionality) - [ ] Improvement (change adding some improvement to an existing functionality) - [ ] Documentation update **How Has This Been Tested** (Please describe the tests that you ran to verify your changes. And ideally, reference `tests`) - [x] Adding new tests covering the feature. **Checklist** - [ ] I added relevant documentation - [ ] follows the style guidelines of this project - [ ] I did a self-review of my code - [ ] I made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] I filled out [the contributor form](https://tally.so/r/n9XrxK) (see text above) - [ ] I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
No description provided.
The text was updated successfully, but these errors were encountered: