Skip to content
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

[XI-6385] Change the set of questions the user can start the quiz with #83

Merged
merged 4 commits into from
May 13, 2024

Conversation

JuliaCasamitjana
Copy link
Collaborator

Description

To make sure the name of the sets (quick, medium, large) are consistent with the number of questions we decided to make the sets fixed.

The user can now choose between 3 fixed sets (10, 20, 50) and the complete set. The complete set is always the option with the most questions. If any of the fixed sets is larger we don't show it.

quiz-recap-intro

Decisions / Choices I made

  • The last two commits improve the test data. We don't need to create new data objects for test subsets. We can
    use the existing testData object and slice it to the required size.

Checklist

  • All checks pass successfully
  • All related commits are squashed together
  • The changes are properly documented / the relevant documentation is updated (if applicable)
  • Appropriate labels are added to this pull request

@JuliaCasamitjana JuliaCasamitjana added the enhancement New feature or request label May 8, 2024
@JuliaCasamitjana JuliaCasamitjana self-assigned this May 8, 2024
src/App.test.tsx Outdated Show resolved Hide resolved
The user can now choose between 3 fixed sets (10, 20, 50) and the complete set.
The complete set is always the option with the most questions. If any
of the fixed sets is larger we don't show it.

Adjust and extend tests.

Closes #82
We don't need to create new data objects for test subsets. We can
use the existing testData object and slice it to the required size.
Copy link
Collaborator

@christophblessing christophblessing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@JuliaCasamitjana JuliaCasamitjana merged commit 51ab3ce into main May 13, 2024
4 checks passed
@JuliaCasamitjana JuliaCasamitjana deleted the jc/6385_change_sets branch May 13, 2024 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants