feat: add a user context provider to the app #24
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MattCMcCoy
commented
Feb 2, 2024
| export default function Router() { | ||
| return ( | ||
| <NavigationContainer> | ||
| <AppNavigation /> |
There was a problem hiding this comment.
Im thinking if onboarding ever gets added it can be its own nav stack. Or if we ever want to have another nav stack can also be separated here
|
Not sure if we should hold off on this, as refactoring app.tsx mid sprint is probably a bad idea |
Found solution to RNP components not utilizing tailwind styling, some changes to the components to make them more generic to allow it to be used in multiple places with different content
MattCMcCoy
commented
Feb 4, 2024
There was a problem hiding this comment.
Just adding a file input into swagger for file upload
MattCMcCoy
commented
Feb 4, 2024
There was a problem hiding this comment.
Basically cleanup, but also adding context and paper provider so everywhere has access to RNP and our group and user context
MattCMcCoy
commented
Feb 4, 2024
There was a problem hiding this comment.
figured out how to get tailwind to work in RNP lol
MattCMcCoy
commented
Feb 4, 2024
There was a problem hiding this comment.
figured out how to get tailwind to work in RNP lol
MattCMcCoy
commented
Feb 4, 2024
There was a problem hiding this comment.
I wanted an example file people could visit for frontend stuff, this was basically my goal with this refactor
andrewcaplan1
approved these changes
Feb 5, 2024
andrewcaplan1
pushed a commit
that referenced
this pull request
Feb 6, 2024
* feat: userContext stuff idk man * refactor: general all around fixes (removing unused imports, fixing navigation errors) * refactor: all over the place with this commit, but added swagger docs for file upload * refactor: make context more general * refactor: utilize card and popup in the medlist FE
oliviaseds
added a commit
that referenced
this pull request
Feb 17, 2024
* Added mock data * feat: get tasks by gid/uid/status/type/date * style: formatting * fix: Remove commitizen branch * test: fix task tests * style: quick reformat * feat: assign user(s) to a task CRUD and test * style: quick style * refactor: abstract queries, enable multiple * refactor: abstract filter query/arg building logic * crud routes for create new task, delete task, and update task info * Feat: generic card and popup modal (#20) * Intial card/button * Basic card and modal Basic card and modal skeleton * fixed card and popup to have the requirments fixed card and popup to have the requirments * wip for passing medication data into Card component * trying to get stuff workin * trying to get comps workin * stuff * Final Working Card + Popup --------- Co-authored-by: AdharshKan42 <[email protected]> * fix: go formatting stuff werent running through entire directory (#23) * feat: add a user context provider to the app (#24) * feat: userContext stuff idk man * refactor: general all around fixes (removing unused imports, fixing navigation errors) * refactor: all over the place with this commit, but added swagger docs for file upload * refactor: make context more general * refactor: utilize card and popup in the medlist FE * backend hot reload * feat: added hot reloading for backend code * refactor: abstract queries, enable multiple * refactor: routes are updated * feat: new route backend * feat: add a new label crud and test * feat: delete label crud and test * refactor: FE remove extra card file * test: fix typo in db, tests passing locally * refactor: modify task filter to bind to query * feat: update task name and color route and test * docs: fix swaggo comment style for task assignment and deletion * docs: same swagger changes for label routes * refactor: implement inclusive data range filter * refactor: keep consistency between route groups * docs: fix a swagger doc return type --------- Co-authored-by: bedrockdude10 <[email protected]> Co-authored-by: haleymartin-6 <[email protected]> Co-authored-by: CaitlinFlynn <[email protected]> Co-authored-by: AdharshKan42 <[email protected]> Co-authored-by: Matt McCoy <[email protected]> Co-authored-by: Matt McCoy <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Link to Ticket
Please include a summary of the changes and the related issue. Please also
include relevant motivation, context, and images!
Added a user context provider to allow user things like id, email and other things in the future to be added globally to components.
Refactored App.tsx to clean up the code a bit and make it more expandable.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. If they are unit
tests, provide the file name the tests are in. If they are not unit tests,
describe how you tested the change.
I printed the stuffs on the main screen
Checklist