-
Notifications
You must be signed in to change notification settings - Fork 184
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 2024-01-01 and 2025-01-01 Regulations to History #10480
Conversation
<li><%= link_to "2025-01-01", "./official/2025-01-01/" %> (<%= link_to "Changes", "https://github.com/thewca/wca-regulations/compare/official-2024-01-01...official-2025-01-01-fixed#files_bucket" %>, <%= link_to "Summarized Changes", "https://www.worldcubeassociation.org/posts/wca-regulations-january-2025" %>)</li> | ||
<li><%= link_to "2024-01-01", "./official/2024-01-01/" %> (<%= link_to "Changes", "https://github.com/thewca/wca-regulations/compare/official-2023-08-01...official-2024-01-01#files_bucket" %>, <%= link_to "Summarized Changes", "https://www.worldcubeassociation.org/posts/wca-regulations-january-2024" %>)</li> | ||
<li><%= link_to "2023-08-01", "./official/2023-08-01/" %> (<%= link_to "Changes", "https://github.com/thewca/wca-regulations/compare/official-2023-02-01...official-2023-08-01#files_bucket" %>, <%= link_to "Summarized Changes", "https://www.worldcubeassociation.org/posts/wca-regulations-august-2023" %>) | ||
<ul> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are you opening a new ul
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Versions of the same year should be displayed in a nested list (for some reason I'm unaware of). See here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, thanks!
LGTM, can this be merged now? (Seeing that 2025-01-01 regs are released) |
Yes! (and please see the latest WRC-WST email). |
!!! Do not merge before 2024-01-01 Regulations and Guidelines are released !!!
@thewca/wrc-team please review in case there's something wrong. It's a simple change, but I couldn't test it because docker-compose is not working for me.
We may need to wait for thewca/wca-regulations#1191 as well.