-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Deep linking to runs on schedule pages #31
Comments
We could also do |
Good point - I agree we should also have |
I don't think linking by any sort of relative ordering is useful because organisers move around runs so much. The use case for deep linking a row is mostly so people can link directly to a specific run in the schedule and we'd want that link to remain valid through all schedule edits. |
I was afraid that was the case. Oh well, it was worth a shot as then the link would have been slightly more meaningful. Consistency is better. |
If @duncte123 would be able to tweak the database, at submission time a "friendly uuid" could be generated and added to the submission row like
Seems a lot of work for a relatively minor feature but it would make the URLs much, much nicer particularly when linked on an instant messenger like Discord. We don't want the URL to expire if any of the run properties are changed so this is something that would need to be locked in permanently when the submission is first created. |
Yeah, I agree. We'll just stick with our existing immutable properties and maybe in the future we can revisit. I made them a little clearer by making the anchor be |
Will be considered resolved after this issue is fixed. #32 (comment) |
The schedule page should have two deep linking functions - individual runs via run UUID and
#now
In page references often suffer from being implemented with no discovery, so we should address that as follows by exposing them to the user in the following places:
rel="nofollow"
as it is circular.rel="nofollow"
The text was updated successfully, but these errors were encountered: