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 ability to update results without affecting existing routes #489

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

hansonwj
Copy link

This functionality will make it possible to have rg2 available at events (not just afterwards).

As competitors finish, admins will be able to upload updated xml files, and routes that have already been uploaded will be renumbered so that they are against the correct competitor. The recently-finished competitors will be able to upload their routes once the xml update has been done.

Alongside pull request 487, uploading routes should be simple on georeferenced maps.

This is the first step - ideally it would be possible to automate this process, however because a lot of parsing is done by the client before it is sent to the server, I've focused on getting it working from the client side first.

You may want to initially call this a 'beta' feature as I have only tested it on relatively 'normal' events.

How it works:

  • The user adds an intial course and results file.
  • The user lists all of the possible permutations of class to course.
  • These class to course mappings are added to the json payload when an event is created and saved in a new file (mappings_.txt)
  • When the event update is requested, the server checks the mappings file to see which courseid to map each result to ("Do not save" will remain ignored)
  • The server then updates the kilpailijat, kommentit and merkinnat files, adding new runners with correct courseid and updating the positions of previously-added runners.

@hansonwj
Copy link
Author

hansonwj commented Aug 1, 2021

Note: currently sets a default value for variants so not recommended if you have variants in your courses. Will have a look at fixing this.

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.

1 participant