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

Add form input dropdown #249

Merged
merged 3 commits into from
Aug 7, 2024
Merged

Add form input dropdown #249

merged 3 commits into from
Aug 7, 2024

Conversation

tdanielles
Copy link
Contributor

@tdanielles tdanielles commented Aug 4, 2024

Description

  • add dropdown for formInput field
  • once selected for a question, it's disabled
  • formInputs that are already selected for other questions don't show up as options when adding new questions
  • add major as hardcoded question alongside school and country

@tdanielles tdanielles changed the base branch from main to dev August 4, 2024 21:05
Copy link

github-actions bot commented Aug 4, 2024

Visit the preview URL for this PR (updated for commit 2532ddc):

https://nw-admin-portal--pr249-add-form-input-dropd-vwb3z6wp.web.app

(expires Wed, 14 Aug 2024 21:56:00 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: d819f9eb3b1b79c03e9109f08eeff92215a2990e

@tdanielles tdanielles requested a review from martincai8 August 5, 2024 05:07
@tdanielles tdanielles marked this pull request as ready for review August 5, 2024 05:07
Copy link
Contributor

@martincai8 martincai8 left a comment

Choose a reason for hiding this comment

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

lgtm again🚀 one small thing—it might be good to indicate that the form input is locked (maybe by greying it out)

@tdanielles tdanielles merged commit 48d909d into dev Aug 7, 2024
2 checks passed
tdanielles added a commit that referenced this pull request Sep 28, 2024
* implemented sidebar and routing

* fix bug

* added general structure and formatting

* implemented welcome page

* draft question component

* janky (but working) code for form

* fix linting

* restore yarn.lock for now

* add yarn.lock back

* styled components pro max + named export for hacker app navbar

* addressed the rest of the comments

* added save button that does absolutely nothing

* fix linting?

* fixed linting fr this time LOL

* addressed comment

* move save btn + added firebase get and save for skills and basic info

* welcome db connect + resolve some warnings

* clean up

* more cleanup

* add lasteditedby and lastedited at

* await handleSave and added padding

* Add form input dropdown (#249)

* initial push

* added major and form input dropdown

* greyed out when selected

* CMS form builder additions (#250)

* Add Questionnaire section, remove Checkbox and File Upload question types, 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

* final

* final + tested

* fix linting

* bring the hacker apps section of the sidebar back

* remove package-lock

* fix lint error

* run ci on PRs to dev

---------

Co-authored-by: tdanielles <[email protected]>
Co-authored-by: martincai8 <[email protected]>
Co-authored-by: Martin Cai <[email protected]>
Co-authored-by: trisha <[email protected]>
Co-authored-by: jkuo630 <[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.

2 participants