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

feat: implemented api documentation, user management and sending notifications #14

Closed
wants to merge 19 commits into from
Closed

feat: implemented api documentation, user management and sending notifications #14

wants to merge 19 commits into from

Conversation

vickywane
Copy link
Contributor

@vickywane vickywane commented May 24, 2021

This pull request is a continuation of this pull request.

A URL mismatch bug between the preview generated by Netlify and the preview URL generated by the wait-for-netlify-action action caused the Wait for successful preview deployment job within the CI to fail here.

The following items were the objectives of the previous pull request:

  • Run tests against a locally deployed version of the API in the current PR

  • Run tests against a netlify preview deployment of the API

  • Add a code coverage report after the test run and semantic release

  • Tag and label a new GitHub release

  • Publish client SDK to npm

Follow up pull requests:

vickywane and others added 17 commits April 2, 2021 21:10
* feat: project CI

* feat: corrected docs and added architecture diagram to readme

* feat: project CI

* update: added code coverage report

* update: added code coverage report

* update: updated function to use local netlify server

* update: updated function to use local netlify server

* update: rebuild CI checks

* update: rebuild CI checks

* update: rebuild CI checks

* update: rebuild CI checks

Co-authored-by: Ivelin Ivanov <ivelin@ambianic.ai>
* feat: project CI

* feat: corrected docs and added architecture diagram to readme

* feat: project CI

* update: added code coverage report

* update: added code coverage report

* update: updated function to use local netlify server

* update: updated function to use local netlify server

* fix: moved stripe key to secrets && improved scripts

* fix: update

* fix: update

Co-authored-by: Ivelin Ivanov <ivelin@ambianic.ai>
…ment (#5)

* feat: project CI

* feat: corrected docs and added architecture diagram to readme

* feat: project CI

* update: added code coverage report

* update: added code coverage report

* update: updated function to use local netlify server

* update: updated function to use local netlify server

* fix: moved stripe key to secrets && improved scripts

* fix: fixed broken github pages dir

Co-authored-by: Ivelin Ivanov <ivelin@ambianic.ai>
* feat: improved user documentation

* feat: improved user documentation

* feat: improved user documentation
* Revert "feat: improved user documentation (#6)"

This reverts commit fec4c09.

* feat: corrected ci steps
* Revert "feat: improved user documentation (#6)"

This reverts commit fec4c09.

* feat: corrected ci steps

* feat: corrected ci steps
### Bug Fixes

* fix breaking CI setup ([9d6ec45](vickywane@9d6ec45))
* fix breaking CI setup ([a8ef5db](vickywane@a8ef5db))
* removed lighthouse checks and updated readme ([49288ba](vickywane@49288ba))

### Features

* added postman API test created with newman to CI ([f12e8ab](vickywane@f12e8ab))
* corrected semantic release steps on gh ci ([#8](https://github.com/vickywane/ambianic-subscriptions/issues/8)) ([c55e72e](vickywane@c55e72e)), closes [#6](https://github.com/vickywane/ambianic-subscriptions/issues/6)
* implemented automatic publishing to github pages on each deployment ([#5](https://github.com/vickywane/ambianic-subscriptions/issues/5)) ([18299ea](vickywane@18299ea))
* improved user documentation ([#6](https://github.com/vickywane/ambianic-subscriptions/issues/6)) ([fec4c09](vickywane@fec4c09))
* improved user documentation ([#7](https://github.com/vickywane/ambianic-subscriptions/issues/7)) ([fb7129e](vickywane@fb7129e)), closes [#6](https://github.com/vickywane/ambianic-subscriptions/issues/6)
* project CI ([42f3999](vickywane@42f3999))
* updated postman tests suites ([6893565](vickywane@6893565))
* Revert "feat: improved user documentation (#6)"

This reverts commit fec4c09.

* feat: corrected ci steps

* feat: corrected ci steps

* feat: corrected ci steps
* feat: corrected ci steps

* feat: corrected ci steps
@commit-lint
Copy link

commit-lint bot commented May 24, 2021

Features

Chore

  • release: 1.0.0 [skip ci]nn# 1.0.0 (2021-05-22) (6dcb149)
  • release: 1.1.0 [skip ci]nn# 1.1.0 (2021-05-22) (d2f01f6)
  • release: 1.2.0 [skip ci]nn# 1.2.0 (2021-05-22) (71c3485)
  • release: 1.3.0 [skip ci]nn# 1.3.0 (2021-05-23) (b002d7a)
  • release: 1.4.0 [skip ci]nn# 1.4.0 (2021-05-23) (0ffa6e6)

Update

  • removed duplicate MiscApi file (c45f9b1)

Contributors

vickywane, semantic-release-bot

Commit-Lint commands

You can trigger Commit-Lint actions by commenting on this PR:

  • @Commit-Lint merge patch will merge dependabot PR on "patch" versions (X.X.Y - Y change)
  • @Commit-Lint merge minor will merge dependabot PR on "minor" versions (X.Y.Y - Y change)
  • @Commit-Lint merge major will merge dependabot PR on "major" versions (Y.Y.Y - Y change)
  • @Commit-Lint merge disable will desactivate merge dependabot PR
  • @Commit-Lint review will approve dependabot PR
  • @Commit-Lint stop review will stop approve dependabot PR

@codecov
Copy link

codecov bot commented May 24, 2021

Codecov Report

❗ No coverage uploaded for pull request base (main@0de83a1). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #14   +/-   ##
=======================================
  Coverage        ?   47.66%           
=======================================
  Files           ?        3           
  Lines           ?      342           
  Branches        ?      108           
=======================================
  Hits            ?      163           
  Misses          ?      179           
  Partials        ?        0           

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