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

Dev #255

Merged
merged 37 commits into from
Sep 28, 2024
Merged

Dev #255

merged 37 commits into from
Sep 28, 2024

Conversation

DonaldKLee
Copy link
Member

No description provided.

tdanielles and others added 30 commits July 14, 2024 10:08
Connect hacker app form to firebase
* initial push

* added major and form input dropdown

* greyed out when selected
…ypes, logic and validation changes (#251)

* remove checkbox, fileupload, and edit logic for formInput

* fix linting issues?

* remove portfolio type from basic info and country, major, name, and school from skills

* add questionnaire section and formInput validation when saving
Copy link

github-actions bot commented Sep 24, 2024

Visit the preview URL for this PR (updated for commit 9db91cb):

https://nw-admin-portal--pr255-dev-6cjzot9z.web.app

(expires Fri, 04 Oct 2024 18:37:07 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: d819f9eb3b1b79c03e9109f08eeff92215a2990e

@martincai8
Copy link
Contributor

martincai8 commented Sep 27, 2024

note: we'll have to fix the flag toggling since we changed the livesite firebase schema, but we can do that later 😎 we can just change the values manually for now

Copy link
Contributor

@jkuo630 jkuo630 left a comment

Choose a reason for hiding this comment

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

o shit thanks for this @martincai8 clutched up fr

Copy link
Contributor

@tdanielles tdanielles left a comment

Choose a reason for hiding this comment

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

goat

@tdanielles tdanielles merged commit deec103 into main Sep 28, 2024
5 checks passed
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.

4 participants