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

Let users set event location before registration type #4033

Conversation

ivarnakken
Copy link
Member

Description

I don't see how the location has anything to do with the registration type. I have personally created an event where I knew the location but not the registration type, so this restriction is just annoying.

Testing

  • I have thoroughly tested my changes.

Tested that it works to create an event with a location but without any specified registration type.

@ivarnakken ivarnakken added review-needed Pull requests that need review bug-fix Pull requests that fix a bug small-fix Pull requests that fix something small labels Jul 12, 2023
@ivarnakken ivarnakken requested a review from a team July 12, 2023 18:37
@ivarnakken ivarnakken self-assigned this Jul 12, 2023
@linear
Copy link

linear bot commented Jul 12, 2023

ABA-384 Let users set event location before "påmeldingstype"

For some reason I have to specify the "påmeldingstype" to be able to set a location for an event. That's kind of dumb.

<Field
label="Sted"
name="location"
placeholder="Den gode nabo, R5, ..."
Copy link
Member Author

Choose a reason for hiding this comment

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

Removed R5 from the placeholder, since that's part of MazeMap.

I don't see how the location has anything to do with the registration type. I
have personally created an event where I knew the location but not the
registration type, so this restriction is just annoying.
@ivarnakken ivarnakken force-pushed the ivarnakken/aba-384-let-users-set-event-location-before-pameldingstype branch from 41bd48d to 34fa687 Compare July 12, 2023 21:43
@ivarnakken ivarnakken requested a review from ollfkaih July 13, 2023 10:01
Copy link
Contributor

@ollfkaih ollfkaih left a comment

Choose a reason for hiding this comment

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

LGTM

@ivarnakken ivarnakken added approved Pull requests that have been approved ready-to-merge Pull requests that have been approved and are ready to be merged and removed review-needed Pull requests that need review labels Jul 13, 2023
@ivarnakken ivarnakken merged commit 0b8286f into master Jul 13, 2023
4 checks passed
@ivarnakken ivarnakken deleted the ivarnakken/aba-384-let-users-set-event-location-before-pameldingstype branch July 13, 2023 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved bug-fix Pull requests that fix a bug ready-to-merge Pull requests that have been approved and are ready to be merged small-fix Pull requests that fix something small
Projects
None yet
2 participants