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.

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

Sync Upstream #152

Merged
merged 13 commits into from
Apr 4, 2024
Merged

Sync Upstream #152

merged 13 commits into from
Apr 4, 2024

Conversation

nickbar01234
Copy link
Collaborator

Description

Sync upstream

  • UI fixes
  • Drive security

nickbar01234 and others added 13 commits March 25, 2024 16:36
…ew senior folder in correct chapter, and update share permissions on new member.
* Add route protection

* Protect uid segment

* Remove comment
* Update UI

* Fix layout props

* Return if no chapter ID is found

* Address comments
* Update pending card style

* Add to DisplayChapterView

* Update dashboard
* styling done, must order.

Co-authored-by: cledi01 <[email protected]>
Co-authored-by: Tyler <[email protected]>
Co-authored-by: Kim Nguyen <[email protected]>
Co-authored-by: Johnny Tan <[email protected]>
Co-authored-by: Won Kim <[email protected]>

* change styling for dropdown position and make role font size smaller

* fix senior tile styling

* small fix

* add sortedStudents function

* add chapter leader and remove president position

* Fix route

---------

Co-authored-by: bellybubjoeOfficial <[email protected]>
Co-authored-by: cledi01 <[email protected]>
Co-authored-by: Tyler <[email protected]>
Co-authored-by: Kim Nguyen <[email protected]>
Co-authored-by: Johnny Tan <[email protected]>
Co-authored-by: Won Kim <[email protected]>
Co-authored-by: wkim10 <[email protected]>
Co-authored-by: nickbar01234 <[email protected]>
Copy link

vercel bot commented Apr 4, 2024

@wkim10 is attempting to deploy a commit to the legacy-project Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Collaborator

@kimhnguyenn kimhnguyenn left a comment

Choose a reason for hiding this comment

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

LGTM

@nickbar01234 nickbar01234 merged commit 9d9f12d into JumboCode:main Apr 4, 2024
1 of 3 checks passed
nickbar01234 added a commit that referenced this pull request Apr 11, 2024
* remove errors when user re-types something into the form

* Use reset helper function

---------

Co-authored-by: wkim10 <[email protected]>
Co-authored-by: nickbar01234 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants