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

Handle 404 errors with react router systems #1231

Merged
merged 13 commits into from
Jan 9, 2025

Conversation

joshuadkitenge
Copy link
Collaborator

@joshuadkitenge joshuadkitenge commented Jan 8, 2025

Description

BaseLayout is implement in this pr #1217

Testing instructions

Add a set up instructions describing how the reviewer should test the code

  • Review code
  • Check Actions build
  • Review changes to test coverage
  • {more steps here}

Copy link

codecov bot commented Jan 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.13%. Comparing base (2101f0c) to head (8d0f9a1).
Report is 19 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1231      +/-   ##
===========================================
+ Coverage    99.11%   99.13%   +0.02%     
===========================================
  Files           94       94              
  Lines        17641    17658      +17     
  Branches      2130     2953     +823     
===========================================
+ Hits         17485    17506      +21     
+ Misses         155      148       -7     
- Partials         1        4       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@joshuadkitenge joshuadkitenge marked this pull request as ready for review January 8, 2025 13:40
@joshuadkitenge joshuadkitenge force-pushed the nested-routes-for-system-#1213 branch from 27c551f to b6345f9 Compare January 8, 2025 16:53
@joshuadkitenge joshuadkitenge force-pushed the nested-routes-for-system-#1213 branch from 9fe5a77 to 2101f0c Compare January 9, 2025 09:48
Base automatically changed from nested-routes-for-system-#1213 to develop January 9, 2025 11:35
Copy link
Collaborator

@joelvdavies joelvdavies left a comment

Choose a reason for hiding this comment

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

Worked well for me.

@joshuadkitenge joshuadkitenge merged commit 89fd618 into develop Jan 9, 2025
6 checks passed
@joshuadkitenge joshuadkitenge deleted the handle-404-errors-with-react-router-systems branch January 9, 2025 12:09
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.

2 participants