Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

docs: add documentation for notifications feature#14478

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
dannykopping merged 9 commits intomainfromdk/notification-docs
Aug 30, 2024

Conversation

dannykopping
Copy link
Contributor

@dannykoppingdannykopping commentedAug 29, 2024•
edited
Loading

Copy link
Collaborator

@stirbystirby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Going to add a little section towards the top about user preferences.

Notifications are sent by Coder in response to specific internal events, such as a workspace being deleted or a user
being created.

**Notifications are currently an experimental feature.**
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Let's call out the release explicitly until our docs have versioning.

**Notifications are currently an experimental feature initially released in [2.15.0](https://github.com/coder/coder/releases/tag/v2.15.0).**

Copy link
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

This is causing CI to fail since the release is not yet present:https://github.com/coder/coder/actions/runs/10629250397/job/29465701530?pr=14478

We can ignore this failure and merge, or wait to merge until the release has been prepared.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

We need it in main before we cut the tag for offline docs, let's just skip the link.

Copy link
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

OK, removed in6a6b903.

dannykoppingand others added5 commitsAugust 30, 2024 09:28
Signed-off-by: Danny Kopping <danny@coder.com>
Signed-off-by: Danny Kopping <danny@coder.com>
@dannykoppingdannykopping changed the title[WIP] notifications docsdocs: add documentation for notifications featureAug 30, 2024
Signed-off-by: Danny Kopping <danny@coder.com>
Signed-off-by: Danny Kopping <danny@coder.com>
@dannykoppingdannykopping marked this pull request as ready for reviewAugust 30, 2024 08:25
Copy link
Member

@johnstcnjohnstcn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Nice documentation 👍

dannykopping reacted with heart emoji
Copy link
Member

@mtojekmtojek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Great documentation!

Signed-off-by: Danny Kopping <danny@coder.com>
@dannykoppingdannykopping merged commit043f4f5 intomainAug 30, 2024
28 checks passed
@dannykoppingdannykopping deleted the dk/notification-docs branchAugust 30, 2024 15:30
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsAug 30, 2024
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@mtojekmtojekmtojek approved these changes

@stirbystirbystirby left review comments

@johnstcnjohnstcnjohnstcn approved these changes

Assignees

@dannykoppingdannykopping

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

docs: write up docs for notifications

4 participants

@dannykopping@johnstcn@mtojek@stirby
[8]ページ先頭
©2009-2025 Movatter.jp