Data setup #74
Merged
Data setup #74
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
Contributor
ciaranschutte
commented
Aug 20, 2023
•
ciaranschutte
commented
- large amount of files are from updating the import statement for App config provider
- adds GQL Apollo handling
- adds typechecking with generated types
- adds auth for handling gql requests
- adds basic loading and error handling - to be improved loading and error improvements #77
- hydration issues are an existing bug 🐛 hydration issues with Suspense boundary in AuthProvider #78
* move global into src, instead of app folder * add some components * configure uikit, emotion and typings * uikit happy * Basic Env Vars * Populating Basic Configs * cleanup * add theme provider * emotion imports * emotion uikit style tpying * First Working Test * Working Redirect w/ URL Join * remove DataCallout uikit re-export wrapper * Basic Reusable Component * Remove .env * Update .gitignore * Remove Duplicate * Stage for UI Kit * Loading Test w/ UI Kit * NavBar Setup * Fix Landing Page * Remove Next logo * Skeleton Cookies * Basic Logout * AuthContext Setup * AuthContext Simplest State * Generic Vars * Major Update w/ Logout + UI Kit * Testing Context + State * Add Copyrights * Update TypeScript, Imports, File Structure * Login Button Component * Use AuthContextValue * Use GoogleLogin * Use URL Join * Local .env updates * Remove Boilerplate * Shuffle Files Around * Refactor w/o Context * Remove Logging * Fix env settings * Add React Query + Mock Login State * Add Copyright * Current Working State * Fix User Badge * Temporary Login Solution * Clean Up State Changes * Remove Redirect * Cleaned Up User Badge * Basic Pathname Solution * Cleaned Up Login Flow * Improved Layout + Header Handling * Remove vscode from gitignore * Fix /public import * Update page names * Refactor Auth / Layout * General Clean Up * Stub Middleware * Tidying * Middleware + Auth Route Setup - Infinite Loop * Current State - Testing * Children prop cleanup * Header UI Updates * Current Working State * Clean Up Use of Stored Token * Remove extra storedToken * First Lint Changes * Hook + Middleware Cleanup * Better fallback, LoggingIn dependency * Move Middleware to Feature Branch * Unnecesary changes * Remove Server side token management * First Setup * Log Out Update * Updated Var name + Conditional * Improve Loading Var Names + Reusable Logout * Use Context logOut * Remove Dummy Logout Button * 1st Header PR Feedback * Remove Unused Imports * Fix Build Issues * ProfileMenu Component * Reusable Login --------- Co-authored-by: Ciaran Schutte <[email protected]> Feat/#14 programs list page (#47) * add submission layout * add some components * add All Programs layout * skeleton * cleanup + add linking for programs * remove sidemenu logic for PR * remove program list and search func for PR * fix type error * mmove types around * fix build * clean import --------- Co-authored-by: Ciaran Schutte <[email protected]> add All Programs layout skeleton cleanup + add linking for programs remove sidemenu logic for PR remove program list and search func for PR use route groups content additions to sidemenu poor mans sidebar toggle Revert "Merge branch 'develop' into sidemenu-layout-restructure" This reverts commit 640bb4b. clean up add sidemenu toggle fix build lots of styling add page count add table comps, fix font fix percent cell table updates sidemenu styling styling cleanup cleanup
Homepage feedback (#68) * fix footer placement links * fix high res layout not expanding data callout * use chevron icon * Update Footer.tsx --------- Co-authored-by: Ciaran Schutte <[email protected]>
remove slice of Programs in comps add sidemenu query
ciaranschutte
force-pushed
the
data-setup
branch
from
2cdb400 to
cfc9da2
Compare
ciaranschutte
commented
Aug 22, 2023
| dotenv.config({ path: '.env.local' }); | ||
|
|
||
| const gqlConfig: CodegenConfig = { | ||
| schema: `${process.env.NEXT_PUBLIC_GATEWAY_API_ROOT}/graphql`, |
There was a problem hiding this comment.
this file is outside the context of Next but it makes sense to try and at least keep consistent naming/file instead of having multiple env configurations depending on process context
ciaranschutte
requested review from
demariadaniel,
justincorrigible and
joneubank
| @@ -0,0 +1 @@ | |||
| export type Programs = any; | |||
There was a problem hiding this comment.
yes! nice catch! removing it as gql has inferred types now
demariadaniel
approved these changes
Aug 22, 2023
Merged
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.