-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
feat: added documentation for premium notifications #28
base: master
Are you sure you want to change the base?
Conversation
Features
UpdateBug FixesContributorsCommit-Lint commandsYou can trigger Commit-Lint actions by commenting on this PR:
|
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.
@vickywane make this PR available for preview on your main fork with all CI checks passing.
Added a few inline comments for language styling.
* feat: added documentation for premium notifications * update: expanded docs and added relevant images * update: linked premium-services to quickstart
Something is not adding up. When I look at your fork, there is no CI
running. No actions triggered.
https://github.com/vickywane/ambianic-docs/actions
…On Wed, May 26, 2021 at 6:10 PM Nwani Victory ***@***.***> wrote:
@ivelin <https://github.com/ivelin> This
<https://github.com/vickywane/ambianic-docs> is my fork of this pull
request available for review.
All CI checks passed in the pull request here
<vickywane#1> before it was merged.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#28 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AARBUFLFMMLBF3JS2A5CKJLTPV5XXANCNFSM45OPJL2A>
.
|
d72c329
to
3ef7697
Compare
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.
@vickywane The text looks good. See comments on organizing sections and menus.
59fa174
to
94eb4ab
Compare
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.
Its looking good. Only one comment left, which I thought I made in the previous review, but I don't see it in the history. Other than that, we can merge this docs PR as soon as the code is ready.
464b954
to
76b2e9c
Compare
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.
It's looking good. Will merge together with the cloud api code when ready.
This pull request adds the documentation of the new premium notifications within the Ambianic PWA UI here into the Ambianic Docs
The new changes are introduced in the
premium-services.md
file which is linked to the Quickstart Guide in the Ambianic Premium Services section.