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

docs: standardise notifications schema #7396

Conversation

ShahanaFarooqui
Copy link
Collaborator

Notifications schema was introduced in the last release (commit 932396e) by @ErikDeSmedt. The last release also had a huge documentation update with PR #6995.

This PR is modifying the notifications schema definitions and json schema according to our latest documentation standard.

Changelog-None.

@ShahanaFarooqui ShahanaFarooqui added this to the v24.08 milestone Jun 14, 2024
@ShahanaFarooqui ShahanaFarooqui requested a review from cdecker as a code owner June 14, 2024 20:27
@ShahanaFarooqui
Copy link
Collaborator Author

@ErikDeSmedt You are not showing up in my reviewers list. Please review the PR.

@ShahanaFarooqui ShahanaFarooqui force-pushed the docs-notifications-schema branch from 6cdf673 to b55434b Compare June 14, 2024 20:47
Copy link
Contributor

@ErikDeSmedt ErikDeSmedt left a comment

Choose a reason for hiding this comment

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

Looks great. Thanks for fixing up this PR.

Thanks for changing the id in the channel_opened message from optional to required. I think this was a mistake from my side.

@ShahanaFarooqui
Copy link
Collaborator Author

ShahanaFarooqui commented Jun 18, 2024

@ErikDeSmedt Can you please ACK and approve the PR if everything is okay?

@ShahanaFarooqui ShahanaFarooqui removed the request for review from cdecker June 19, 2024 19:39
@ShahanaFarooqui
Copy link
Collaborator Author

ACK b55434b

@ShahanaFarooqui ShahanaFarooqui force-pushed the docs-notifications-schema branch from b55434b to a9b6bf7 Compare June 19, 2024 19:41
@ErikDeSmedt
Copy link
Contributor

Ack b55434b

@rustyrussell rustyrussell force-pushed the docs-notifications-schema branch from a9b6bf7 to b0c5fab Compare June 21, 2024 04:08
@rustyrussell
Copy link
Contributor

Trivial rebase for genfile conflict.

@ShahanaFarooqui ShahanaFarooqui merged commit 51e4633 into ElementsProject:master Jun 27, 2024
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants