Skip to content

Conversation

DarianRushworth
Copy link
Collaborator

On Merge of all major components, I did a sweep to see if functionality was correct and firing properly across server and client.
Whilst doing this I saw the opportunity to create the "random" exercises category.
What has been implemented:

  • A new page with protected route "/exercise/random"
  • In state all the questions are fetched from the backend
    -As much as the rendering of each question was very simple as of the two completed components already:
  1. Quiz Cards
  2. Quiz Codes
  • Thanks to Diego's Logic on the elements page switching between the components has never been easier.

Copy link
Collaborator

@StephBerg86 StephBerg86 left a comment

Choose a reason for hiding this comment

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

Whoohoo random is also working! So cool how you did this logic. impressive.

Copy link
Collaborator

@DiegoOTdC DiegoOTdC left a comment

Choose a reason for hiding this comment

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

Hi man! Great way to make this work! Just 1 little thing, in the store/exercise/actions, the first dispatch needs to be dispatch(appLoading()) -> you're dispatching appDoneLoading from the start. For the rest really nice man!

@DarianRushworth
Copy link
Collaborator Author

Hi man! Great way to make this work! Just 1 little thing, in the store/exercise/actions, the first dispatch needs to be dispatch(appLoading()) -> you're dispatching appDoneLoading from the start. For the rest really nice man!

Great find, edit complete and successful.

Copy link
Collaborator

@DiegoOTdC DiegoOTdC left a comment

Choose a reason for hiding this comment

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

👍

@DarianRushworth DarianRushworth merged commit 1dd5413 into development Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants