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

[NEW] Add Event Organizer Details on Event Create Form #89

Open
3 tasks
Dnouv opened this issue Jan 19, 2023 · 6 comments
Open
3 tasks

[NEW] Add Event Organizer Details on Event Create Form #89

Dnouv opened this issue Jan 19, 2023 · 6 comments
Assignees
Labels
milestone-1 Issues for first phase of roadmap

Comments

@Dnouv
Copy link
Member

Dnouv commented Jan 19, 2023

Add a new section in the "Event create" form to take the event organizer details input from the user.
Tasks to be done:

  • New form with fields as mentioned above
  • Logic to POST this data into Open Event Server, please refer to this for the required schema of Request
  • Make an Edit component to edit this information later on

Thank you!

Sample Image:
Screenshot_20230119_122753

@Aashray446
Copy link
Contributor

@Dnouv Can I take up this opportunity, I have already backported one of the components and got a very good understanding of the code base. If there is a green signal from you, I can start collecting information in it.

@Dnouv
Copy link
Member Author

Dnouv commented Jan 20, 2023

Once your currently opened PR is approved and merged, you can go ahead and move forward with this. Thank you!

@Aashray446
Copy link
Contributor

Once your currently opened PR is approved and merged, you can go ahead and move forward with this. Thank you!

OK 😄 Thank you!

@Aashray446
Copy link
Contributor

@Dnouv I came up with the following approach :

  • Making a functional component called organizer details that takes handleChange function as a props (It will be inside the EventForm Component, just making it separate to make it more modular and readable, since the form size is big)
  • Add that component to the EventForm component
  • There will be a toggle that shows/hides the organizer details inputs
  • Update the post logic and edit logic, two additional parameters need to handle i.e Owner-name and Owner-description

@Aashray446
Copy link
Contributor

@dnou This is how it looks currently, If you are ok with the UI, I will proceed further

issue-2

@Aashray446
Copy link
Contributor

@Dnouv Can you checkout this PR? 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
milestone-1 Issues for first phase of roadmap
Projects
None yet
Development

No branches or pull requests

2 participants