-
Notifications
You must be signed in to change notification settings - Fork 187
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
Correct issues on events page #1718
Correct issues on events page #1718
Conversation
Final merge and publication blocked by #1698 but still good to go and review regardless in the interim. |
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.
Simple PRs are the best, LGTM
Thanks, we still need to handle CI/CD problems in another issue before merged. |
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.
LGTM as well
I am going to merge this to test in conjunction with #1721. If the pages don't deploy I will be forced to work through it. |
What a surprise, it didn't work. Away we go, more debugging to come and notes. |
Committer Notes
Correct typo and fix links for #1711.
All Submissions:
"?
By submitting a pull request, you are agreeing to provide this contribution under the CC0 1.0 Universal public domain dedication.
Changes to Core Features:
Have you written new tests for your core changes, as applicable?Have you included examples of how to use your new feature(s)?