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

[WCAT] Improvement of the competition form #3759

Open
7 of 11 tasks
DanielEgdal opened this issue Dec 30, 2018 · 16 comments
Open
7 of 11 tasks

[WCAT] Improvement of the competition form #3759

DanielEgdal opened this issue Dec 30, 2018 · 16 comments
Labels
META: good second issue More involved than first issue, these help get a deeper understanding of the codebase project: competition form TEAM: wcat Issues related to WCAT

Comments

@DanielEgdal
Copy link
Member

DanielEgdal commented Dec 30, 2018

Relevant email thread titled "WCAT wish list": https://groups.google.com/a/worldcubeassociation.org/d/msg/software/VtYKpglRpJ8/nQ4KJi7bCQAJ

WCAT would like to have the following updates to the competition form page to avoid issues from Delegates and organisers.

  • Make it impossible to confirm a competition before a schedule on the WCA Website is filled out. (Check schedule and events/rounds match before confirming a competition #3207)
  • Always have all event information be listed before the competition can be confirmed. (This is to ensure the amount of rounds per event appears on the events page as well as the advancement conditions.) (related to Check schedule and events/rounds match before confirming a competition #3207)
  • Give warnings when the cutoff is set to be equal to or higher than the time limit.
  • Make the “Reason for the name” box be mandatory to fill out before submitting a competition.
  • Make it impossible to submit a competition which is less than 28 days from the scheduled start. (This is per the new regulations of 2019.)
  • Remove the markdown text in the schedule tab which refers to the venue name. Markdown issues in the schedule tool #3634, Flatten markdown before putting into schedule #3656
  • Create a standard form which allows WCAT to press a button on the website which sends out an automated message to all competitors to inform them of a newly added event. (I believe this message would be similar to when results of a competition is posted or the ones organisers receive once we announce their competition. When competitors are to respond to this mail, it should make the organiser(s) and Delegate(s) the default receiver, so they can easier update their registration to contain this event.)
  • Only show the nickname field if the name is longer than 32 characters. (WCAT is already ensuring that all names at 32 or less characters have the same name and nickname.)
  • Make it possible to view the events of nearby competitions in the nearby competitions field by a click. (I believe this is similar to the "edit/registrations?show_events=true" page of competitions. This is to easier compare the events of the competitions which have a conflict due to the proximity policy.)
  • Have the ID field only be editable on the admin page. Otherwise it is always the same as the nickname.
  • Show 27 days and below to the competition on the admin page as red. Currently it shows 28 days and below until the competition with red which causes some confusion as we accept competitions with 28 days to the competition.

Best,
Daniel & WCAT

@EdHollingdale
Copy link
Contributor

EdHollingdale commented Dec 31, 2018

In the draft competition announcement policy the the nickname is a "should" not a "must" meaning it can be different (and I believe should be to ensure consistency for world and continental championships). Surely having it auto-populate if the name is less than 32 characters solves most issues (I believe this occurs already when creating a competition)? The same should apply to the competition ID.

@AlbertoPdRF
Copy link
Contributor

AlbertoPdRF commented Dec 31, 2018

I was going to edit the first comment to include more related issues, but I will for now reference them separately as the list is bigger than what I was expecting 😛

I also left comments on some related issues that I believe should be closed instead of including them here.

@jamesquinn1
Copy link

This has been allowed in the past (or has happened at least). Would this no longer be allowed?
Personally I am for not allowing multiple National (or Continental/World) Championships in one year, but idk if consistency would be an issue.

@viroulep
Copy link
Contributor

viroulep commented Jan 4, 2019

This has been allowed in the past (or has happened at least).

I was curious so I looked it up, it happened in Brazil in 2014 (here, and here), I'll comment in the relevant issue.

@jfly jfly added the META: good second issue More involved than first issue, these help get a deeper understanding of the codebase label Jan 15, 2019
@jamesquinn1
Copy link

  • Only show the nickname field if the name is longer than 32 characters. (WCAT is already ensuring that all names at 32 or less characters have the same name and nickname.)

Why can't competitions with names shorter than that have nicknames?

@Jambrose777
Copy link
Contributor

Jambrose777 commented Jan 23, 2019

Addition:

@Epride
Copy link
Member

Epride commented Feb 18, 2019

  • Make the competitor limit required.

Wouldn't this become an issue with multi-location FMC competitions which generally don't specify a limit?

@DanielEgdal
Copy link
Member Author

  • Make the competitor limit required.

Wouldn't this become an issue with multi-location FMC competitions which generally don't specify a limit?

Yes it would be an issue with those competitions, but I assume a simple extenstion to #3755 would fix this issue for the competitor limit as well.

@jfly
Copy link
Contributor

jfly commented Apr 28, 2019

There's a ton of stuff in here, but @Mjic99 will start looking into it after #4069.

@AlbertoPdRF
Copy link
Contributor

Yet another addition:

At this point, I think it would make the most sense to convert this issue into a project!

@Jambrose777
Copy link
Contributor

Jambrose777 commented Jul 26, 2019

  • Add similar regulation check box for asking for restrictions to combination of events.

@AlbertoPdRF
Copy link
Contributor

I've created a project for this issue (as it has many standalone requests) as well as for all the issues listed within it, as it has grown so big that I don't see any other way to manage it 😛 You can see it on: https://github.com/thewca/worldcubeassociation.org/projects/8

@viroulep
Copy link
Contributor

viroulep commented Aug 8, 2019

FWIW I think the overall "competition edit" page would be a good candidate to be ported to js/react given the amount of interaction there is now.

@AlbertoPdRF
Copy link
Contributor

FWIW I think the overall "competition edit" page would be a good candidate to be ported to js/react given the amount of interaction there is now.

Should we move this to its own issue? It seems like a big task to me.

@viroulep
Copy link
Contributor

viroulep commented Aug 8, 2019

oh for sure it's big!
This is probably for after we've done some software "cleaning" steps for the website redesign.

@jfly
Copy link
Contributor

jfly commented Aug 8, 2019

For the record, we currently leverage a lot of magical integrations between activerecord validations and simple_form. Moving the competition edit page to js/react would mean we'd lose all of that.

I'm not sure I'm convinced that there's enough fancy interaction on the competition edit page that it's worth porting to react. The one thing I can think of is the nearby competitions check, which is pretty janky, and would definitely be clearer in pure js. Maybe it would be possible to divide the edit page up into multiple smaller forms? That might help with maintainability and also make it easier to port parts to react if we decide it's worth it.

Jambrose777 pushed a commit to Jambrose777/worldcubeassociation.org that referenced this issue Aug 16, 2019
Jambrose777 pushed a commit to Jambrose777/worldcubeassociation.org that referenced this issue Aug 17, 2019
AlbertoPdRF pushed a commit that referenced this issue Aug 17, 2019
@Jambrose777 Jambrose777 added the TEAM: wcat Issues related to WCAT label Jun 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
META: good second issue More involved than first issue, these help get a deeper understanding of the codebase project: competition form TEAM: wcat Issues related to WCAT
Projects
Status: No status
Status: No status
Development

No branches or pull requests

9 participants