Skip to content

Broadcast form grouping ease of life features - #20505

Merged
ornicar merged 16 commits into
lichess-org:masterfrom
allanjoseph98:relayTourForm-groups-rework
May 28, 2026
Merged

Broadcast form grouping ease of life features#20505
ornicar merged 16 commits into
lichess-org:masterfrom
allanjoseph98:relayTourForm-groups-rework

Conversation

@allanjoseph98

Copy link
Copy Markdown
Member

Re-do the broadcast form for grouping:

Groups

  1. Split the groups form from one single textArea into individual fields for name and tours. This is an API change from
{ info: string }

to

{ info: { name: string, tours: string } }
image
  1. Use tagify on the tours field. When the user focuses on the field, use /api/broadcast/by/{username} to fetch the user's 20 most recent broadcasts.
  2. Filter the broadcasts for tournaments they can actually group depending on if they have StudyAdmin or Broadcaster perms. Set the tagify whitelist accordingly.
  3. Users can fuzzy search the whitelist by tournament name and select broadcasts to add to the group.
  4. If they want to add a broadcast that is not in the whitelist, they can still paste a link. On pasting the link, use /api/broadcast/{broadcastTournamentId} to fetch tournament details. If they fulfil the same criteria to group the broadcast, add it as a valid tag. Failing the check means the tag goes red and they can hover over to see the error. (Either insufficient perms or bad link/broadcast ID). image
  5. Use sortableJS to allow dragging tags around to re-order groups.

ScoreGroups

  1. Split the form from one single textArea into an array of textArea. This is an API change from
{ scoreGroups: string }

to

{ scoreGroups[]: string }

where every element in the array is a new scoreGroup. This is to avoid the confusing comma delimited tourIds and the newline delimited scoreGroups.
2. Use this to show each scoreGroup as its own textArea input. Users can add more scoreGroups by clicking the plus button.
3. Use tagify on these inputs to dynamically set their whitelist to the groups input.
image

AI use disclosure:

Used to fix tagify styling on firefox.

@ornicar
ornicar force-pushed the relayTourForm-groups-rework branch from b7f3472 to 5e338c3 Compare May 27, 2026 07:15
@ornicar

ornicar commented May 28, 2026

Copy link
Copy Markdown
Collaborator

the tagify view isn't responsive and causes mobile view to extend horizontally beyond the screen
image

@ornicar
ornicar force-pushed the relayTourForm-groups-rework branch from 435e5d8 to 6541b60 Compare May 28, 2026 10:35
@ornicar
ornicar merged commit 23ce819 into lichess-org:master May 28, 2026
7 checks passed
@allanjoseph98
allanjoseph98 deleted the relayTourForm-groups-rework branch June 2, 2026 13:31
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.

2 participants