Skip to content

Looking For multiselect dropdown#151

Merged
nurmerey merged 6 commits intocoding-allies:masterfrom
nadyafebi:looking-for-dropdown
Oct 28, 2020
Merged

Looking For multiselect dropdown#151
nurmerey merged 6 commits intocoding-allies:masterfrom
nadyafebi:looking-for-dropdown

Conversation

@nadyafebi
Copy link
Copy Markdown
Contributor

Change the Looking For field of the Project Form to use Creatable from react-select. Closes #100.

Changes

  • Added react-select as dependency.
  • Change the field to use Creatable component.
  • Added new function handleCreatableChange to help handle changes from Creatable component.
  • Modified the validateForm function to handle null value.
  • Modified the handleSubmit function to format the field value from array to comma-separated string.
  • Style the field as best as I can to look similar to the other fields.

Demo

@nadyafebi nadyafebi changed the title Looking for dropdown Looking For multiselect dropdown Oct 2, 2020
Copy link
Copy Markdown
Contributor

@nurmerey nurmerey left a comment

Choose a reason for hiding this comment

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

Asked for a few minor changes! Otherwise LGTM 🙌 ! Thank you @nadyafebi

@nadyafebi
Copy link
Copy Markdown
Contributor Author

@nurmerey I changed the stuff you mentioned 😄

Copy link
Copy Markdown
Contributor

@nurmerey nurmerey left a comment

Choose a reason for hiding this comment

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

One more ask, to remove yarn.lock file. Thank you!

@nadyafebi
Copy link
Copy Markdown
Contributor Author

@nurmerey Removed the yarn.lock (not sure how it got committed there) and also resolved some merge conflict 😃

@nadyafebi
Copy link
Copy Markdown
Contributor Author

Hi @nurmerey @ctran14, I saw someone else got assigned to issue #100. Should I just close this PR then? Thanks

@nurmerey
Copy link
Copy Markdown
Contributor

Hi @nurmerey @ctran14, I saw someone else got assigned to issue #100. Should I just close this PR then? Thanks

Hey sorry for the mixup, the PR looks good. I will approve and merge it (and reassign the task for you!).

Copy link
Copy Markdown
Contributor

@nurmerey nurmerey left a comment

Choose a reason for hiding this comment

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

LGTM 💐 Thank you for great work!

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.

Project Form: Add multi select drop down for 'Looking for' - contributor type (PM, Front end.....)

2 participants