completed addition and route constants #179
Conversation
EmmanuelObi
requested review from
dprincecoder,
zakariyahrobiat and
Mudi-Igbinoba
✅ Deploy Preview for i4g ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for curious-ganache-33d2b4 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
|
@EmmanuelObi please fix your merge conflict we need to close this issue |
fixed and updated the PR. |
|
Good to merge |
|
@EmmanuelObi look into this |
I've fixed the conflict, again. |
|
Kindly fix the merge issue. |
I've done this twice already. Please merge in this time, thanks |
|
@t-bello7 @dprincecoder @ingressive4good I've fixed arising merge conflicts. Please approve and merge in. |
|
Good to merge |
|
👍Good to merge |
There was a problem hiding this comment.
Sorry @EmmanuelObi other PR has been merged, and app.jsx and routes.jsx are now in conflict with yours, kindly fix it again, so this can be merged
| export const NOTIFICATIONS = 'notifications'; | ||
| export const PROFILE = 'profile'; | ||
| export const ADMIN = 'admin'; | ||
| export const USERS = 'users'; |
There was a problem hiding this comment.
I see what you do here good anyways.
🛠️ Fixes Issue
Closes issue #178
✔️ Check List (Check all the applicable boxes)