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.

Sign up for GitHub

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

Jump to bottom

rc/0.5.0 #64

Merged
merged 7 commits into from
Aug 10, 2023
Merged

rc/0.5.0 #64

merged 7 commits into from
Aug 10, 2023

Conversation

ciaranschutte
Copy link
Contributor

@ciaranschutte ciaranschutte commented Aug 10, 2023 •
edited
Loading

demariadaniel and others added 6 commits August 10, 2023 11:14
@demariadaniel @justincorrigible
Feat/8 Logout (#39)
95f7e71 
* 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]>
@ciaranschutte @justincorrigible
Feat/#14 programs list page (#47)
7eeb551 
* 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]>
@demariadaniel @justincorrigible
Add NEXT_PUBLIC Prefix (#52)
8c4a490
@demariadaniel @justincorrigible
Remove Ego Values (#56)
0a755e5
@ciaranschutte @justincorrigible
Runtime client vars (#61)
43b21f6 
* ugprade next -patch

* add api route

* move guts of App into component

* run time vars from server

* cleanup

* use new config

* fix app config with urls in components

* remove getAppCOnfig call in egoJwt lib func

* cleanup config type

* Update README.md

* use npm not npx, so we have our node env set

---------

Co-authored-by: Ciaran Schutte <[email protected]>
@ciaranschutte @justincorrigible
upgrade node to latest LTS version (#63)
bd7c9dd 
Co-authored-by: Ciaran Schutte <[email protected]>
justincorrigible
justincorrigible reviewed Aug 10, 2023
View reviewed changes
package.json
"private": true,
"scripts": {
"dev": "next dev",
"build": "next build",
"build": "NEXT_IS_BUILDING=true next build",
Copy link
Member

Choose a reason for hiding this comment

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

ooh new flag. they just love polluting the env, eh?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that's me polluting the env ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

and yes I do enjoy it :p

@ciaranschutte ciaranschutte marked this pull request as ready for review August 10, 2023 15:32
@ciaranschutte ciaranschutte merged commit 123c16b into main Aug 10, 2023
@ciaranschutte ciaranschutte deleted the rc/0.5.0 branch August 10, 2023 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joneubank joneubank joneubank approved these changes

@justincorrigible justincorrigible Awaiting requested review from justincorrigible

@demariadaniel demariadaniel Awaiting requested review from demariadaniel

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ciaranschutte @justincorrigible @joneubank @demariadaniel