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 event org form field #71

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

r3yc0n1c
Copy link
Contributor

resolves #67

Summary:

  • added a new input field labeled Event Organization (named owner-name) inside the EventBasicCreate component
  • updated the respective state change to show it on the EventShow Page

Screenshots:

Original Updated
ss-original ss-updated
Original Updated
ss-original ss-updated

@Dnouv
Copy link
Member

Dnouv commented Dec 31, 2022

Hey @r3yc0n1c

While you are at improving the UI of the Event Poster page, can you also fix this issue? It occurs when the event description is short (around a few words)

Error Image (as seen on Desktop Screen):

(No need to open a separate issue, push fix changes in this PR only since all of these are minor changes)

Thank you!

@r3yc0n1c
Copy link
Contributor Author

r3yc0n1c commented Jan 3, 2023

Hey @r3yc0n1c

While you are at improving the UI of the Event Poster page, can you also fix this issue? It occurs when the event description is short (around a few words)

Error Image (as seen on Desktop Screen):

(No need to open a separate issue, push fix changes in this PR only since all of these are minor changes)

Thank you!

Yup... I've fixed the UI... let me know if you need any updates
Thanks!

@Dnouv
Copy link
Member

Dnouv commented Jan 28, 2023

Hey @r3yc0n1c
Please resolve the merge conflicts, and add a new checkbox, "Enable Video Room" (the schema is is-videoroom-enabled). Once done, we will be good to merge.

Thank you!

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.

[NEW] Add 'Event Organization' - a new form field
2 participants