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

WIP: New frontend #663

Merged
merged 39 commits into from
Feb 8, 2024
Merged

WIP: New frontend #663

merged 39 commits into from
Feb 8, 2024

Conversation

n8kim1
Copy link
Contributor

@n8kim1 n8kim1 commented Aug 6, 2023 •
edited
Loading

Mega-PR for the new frontend

@acrantel acrantel force-pushed the frontend2 branch 2 times, most recently from 8a352d3 to 1e17ddf Compare August 15, 2023 23:39
@acrantel
Copy link
Member

Note: After we merge this PR, we'll need to regenerate the API because the backend in main is different from the backend in frontend2

@acrantel acrantel marked this pull request as ready for review October 7, 2023 15:03
acrantel
acrantel reviewed Oct 7, 2023
View reviewed changes
.vscode/settings.json Outdated Show resolved Hide resolved
@n8kim1 n8kim1 marked this pull request as draft October 24, 2023 20:22
acrantel and others added 23 commits February 8, 2024 16:46
@acrantel
set up new frontend
37aec27
@acrantel
implement most of routing and add stubs
ff3be85
@lowtorola @acrantel
Api/Auth (#652)
2a36c6e 
Yayy API!!
@acrantel
Add frontend lint / tests to ci workflow (#655)
db0b68d
@lowtorola @acrantel
Table Component/Rankings Page (#658)
b35e8ce 
Co-authored-by: Serena Li <serena.li.usa@gmail.com>
@lowtorola @acrantel
Api Cleanup (#664)
e52d373 
Use functions for api, not static class methods 👍
@acrantel
Add state for current user (#665)
4377723
@acrantel
Upgrade prettier, add some ui elements, add the Register page. (#670)
c30bb1f
@acrantel
Change api generation (#667)
8d47b09
@acrantel
Add login page (#672)
8baa626
@apollo1291 @acrantel
Header and minor ui changes (#654)
3bc246c 
Co-authored-by: Serena Li <serena.li.usa@gmail.com>
@apollo1291 @acrantel
Add DocumentationPage component (for Resources / Quickstart page) and…
61267c4 
… make header/sidebar fixed on scroll (#662)

Co-authored-by: Serena Li <serena.li.usa@gmail.com>
@acrantel
Add tooltip (#673)
0e1c549
@acrantel
Add Collapsible (#674)
441f8e7
@lowtorola @acrantel
Queue Page (#675)
5f045ce 
Co-authored-by: Serena Li <serena.li.usa@gmail.com>
@acrantel
Add team context, update user/episode context (#679)
054d4f6
@n8kim1 @acrantel
Update docs enough for new dev onboarding (#649)
c5e31e0
@acrantel
Move api gen to frontend2, add back old frontend prettier, bump CI no…
ac84ffd 
…de version (#682)
@lowtorola @acrantel
Scrimmaging Page (#677)
d9bb5af 
asdf whooooo
@nicolexxuu @acrantel
Nicole/register kerb field (#692)
b723a3e
@sri240 @acrantel
Add team sidebar display conditions (#689)
5ccc2a2
@n8kim1 @acrantel
Add and organize docs about emails (#683)
e520dbe
@acrantel
Add the my team settings page (#680)
329829b
n8kim1 and others added 15 commits February 8, 2024 16:46
@n8kim1 @acrantel
Add notes from onboarding meeting (#690)
804c365 
Co-authored-by: Serena Li <serena.li.usa@gmail.com>
@lowtorola @acrantel
Tournaments Page (#678)
3757838 
Hooray!
@lowtorola @acrantel
Table Improvements (#697)
6dd4f74 
💰
@acrantel
Make leave team button work, add modal component (#696)
35c3d66
@lowtorola @acrantel
Episode Switching (#698)
383499c 
Co-authored-by: Serena Li <serena.li.usa@gmail.com>
@acrantel
Fix EpisodeSwitcher diff # of hooks rendered bug
7ad2bcc
@nicolexxuu @acrantel
Add submissions page (#701)
cc4171d
@acrantel
add updateteampartial api call, add refreshTeam to team context (#699)
ef53adb
@acrantel
Support changing eligibility and scrim settings (#700)
2ffcfe2
@lowtorola @acrantel
Submission Uploads (#703)
96cd6d4 
Co-authored-by: Serena Li <serena.li.usa@gmail.com>
@nicolexxuu @acrantel
Add the user account settings page (#708)
1896f19
@sri240 @acrantel
Sri/jointeam (#705)
a17d58d
@lowtorola @acrantel
React Query (#707)
d268425 
Co-authored-by: Serena Li <40174697+acrantel@users.noreply.github.com>
@acrantel
Fix routing (#733)
6a62bc6
@lowtorola @acrantel
Home Page (#740)
f188c37 
gahh i cant believe I'm merging this ugly time display hahaha
@acrantel acrantel marked this pull request as ready for review February 8, 2024 22:09
lowtorola
lowtorola approved these changes Feb 8, 2024
View reviewed changes
Copy link
Contributor

@lowtorola lowtorola left a comment

Choose a reason for hiding this comment

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

LGTM!! 🎉

sri240
sri240 approved these changes Feb 8, 2024
View reviewed changes
Copy link
Contributor

@sri240 sri240 left a comment

Choose a reason for hiding this comment

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

flawless

@acrantel acrantel merged commit c6c1052 into main Feb 8, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@acrantel acrantel acrantel left review comments

@sri240 sri240 sri240 approved these changes

@lowtorola lowtorola lowtorola approved these changes

@nicolexxuu nicolexxuu Awaiting requested review from nicolexxuu

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

Successfully merging this pull request may close these issues.
6 participants
@n8kim1 @acrantel @sri240 @lowtorola @apollo1291 @nicolexxuu