-
Notifications
You must be signed in to change notification settings - Fork 197
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
Fix 8.8 and 8.7 announcements links #5152
Conversation
👋 🤖 🤔 Hello, @mesellings! Did you make your changes in all the right places? These files were changed only in docs/. You might want to duplicate these changes in versioned_docs/version-8.6/.
You may have done this intentionally, but we wanted to point it out in case you didn't. You can read more about the versioning within our docs in our documentation guidelines. |
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.
This definitely got tangled up - thank you for the alignment! Good things to think about with the announcements restructuring - these are a lot of places to keep updated and in sync (and currently done twice over for the 8.7/8.8 split!).
- [Web Modeler Deploy diagram change](/reference/announcements/870.md#web-modeler-deploy-diagram-change) | ||
- [Southwest Asia for SaaS region](/reference/announcements/870.md#southeast-asia-region-for-saas-customers-saas) | ||
- [Web Modeler](/reference/announcements/870.md#web-modeler) | ||
- [Southeast Asia region for SaaS customers](/reference/announcements/870.md#southeast-asia-region-for-saas-customers-saas) |
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.
Thank you for fixing this east/west change!!
@akeller @conceptualshark did you want me to do a follow-up PR to change the heading anchor links to use manual links? I was going to here but wasn't sure if these were linked to already, so didn't want to break any existing links? |
@mesellings 😬 Can you create a separate issue for this so we can discuss the concerns with breaking existing links? My immediate concern is the blog, but I don't know how granular those links are and we can request a fix/update from that team. |
See #5168 |
Description
Fixes and aligns the mini-toc links on the 8.7 and 8.8 announcements pages, as per #5067.
When should this change go live?
bug
orsupport
label)available & undocumented
label)hold
label)low prio
label)PR Checklist
/docs
directory (version 8.8)./versioned_docs/version-8.7/
directory (version 8.7)./versioned_docs
directory.@camunda/tech-writers
unless working with an embedded writer.