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

notifications: copy current notif docs #26

Merged
merged 2 commits into from
Jun 7, 2024
Merged

notifications: copy current notif docs #26

merged 2 commits into from
Jun 7, 2024

Conversation

l-zeuch
Copy link
Collaborator

@l-zeuch l-zeuch commented Jun 6, 2024

Copy the currently notifications docs; internal discussion concluded
that this is the current milestone, such that we have a faster migration
to the statically built site.

Some modifications such as notice shortcodes were necessary for it to
work at all. I also took the liberty and removed links to dead /
redundant / not yet existing links.

Signed-off-by: Luca Zeuch [email protected]

Terms

l-zeuch added 2 commits June 6, 2024 21:03
Copy the currently notifications docs; internal discussion concluded
that this is the current milestone, such that we have a faster migration
to the statically built site.

Some modifications such as notice shortcodes were necessary for it to
work at all. I also took the liberty and removed links to dead /
redundant / not yet existing links.

Signed-off-by: Luca Zeuch <[email protected]>
@SoggySaussages
Copy link
Contributor

I've been linking the not yet existing links in my commits thus far, should I stop?

@l-zeuch
Copy link
Collaborator Author

l-zeuch commented Jun 6, 2024

Nah it's alright. I just removed those because I wasn't sure where they'll reside in the final doc.

Come to think of it, having dead links makes Hugo emit a warning during build, so it might be actually sensible to just keep these dead links.

@l-zeuch l-zeuch merged commit 0ab0c09 into master Jun 7, 2024
4 checks passed
@l-zeuch l-zeuch deleted the notifications branch June 7, 2024 11:10
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.

2 participants