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

feature-8782: Add type Yes/No custom from field #8893

Merged
merged 6 commits into from
Jun 16, 2023
Merged

feature-8782: Add type Yes/No custom from field #8893

merged 6 commits into from
Jun 16, 2023

Conversation

khangon
Copy link
Contributor

@khangon khangon commented Jun 13, 2023 •
edited
Loading

Fixes #fossasia/open-event-frontend#8782

Short description of what this resolves:

  • Add type Yes/No custom from field

Changes proposed in this pull request:

  • Add type Yes/No custom from field

Checklist

  • I have read the Contribution & Best practices Guide and my PR follows them.
  • My branch is up-to-date with the Upstream development branch.
  • The unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • All the functions created/modified in this PR contain relevant docstrings.

@auto-label auto-label bot added the feature label Jun 13, 2023
@khangon khangon marked this pull request as ready for review June 15, 2023 09:56
@ghost ghost assigned cweitat and norbusan Jun 15, 2023
cweitat
cweitat previously approved these changes Jun 16, 2023
View reviewed changes
@cweitat cweitat merged commit 5e52f76 into fossasia:development Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@norbusan norbusan norbusan left review comments

@cweitat cweitat cweitat approved these changes

Assignees

@norbusan norbusan

@cweitat cweitat

Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@khangon @norbusan @cweitat @odkhang