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

33 show progress #36

Merged
merged 6 commits into from
Jul 15, 2021
Merged

33 show progress #36

merged 6 commits into from
Jul 15, 2021

Conversation

fadeomar
Copy link
Contributor

Issues this PR relates to

#33

NOTE: REMEMBER NOT TO REVIEW UNLESS ALL ACCEPTANCE CRITERIA HAS BEEN TICKED OFF


Any info the reviewer needs to know to test

e.g. Add env vars, run npm install


Any info the team needs to consider before merged into development

e.g. Will a database need to be rebuilt (and is this ok if existing data), add env vars


Quick Pull Request Checklist

Please make sure you ensure the following has been done before marking your PR as awaiting-review. For full checklist and process review, please refer to Process Guide (#8)

  • Branch is up to date with develop
  • Project runs
  • You can download and install locally, and run the project locally
  • All existing and new features run without bugs
  • Checked that features/design works across all agreed browsers + devices
  • The new work has been checked with Axe accessibility checker and doesn't throw significant errors
  • Linter is used
  • Code is consistent with agreed team style
  • Naming of files is consistent with project
  • Tests all pass
  • No console logs

@ashatat ashatat merged commit b0608dc into develop Jul 15, 2021
@ashatat ashatat deleted the 33-show-progress branch July 15, 2021 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants