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

NOSSUE - Sync API Dcoumentation #1754

Merged
merged 5 commits into from
Apr 17, 2023

Conversation

rodneyosodo
Copy link
Member

Signed-off-by: b1ackd0t blackd0t@protonmail.com

What does this do?

Sync the swagger API documentation with what is implemented on the HTTP API layer

Which issue(s) does this PR fix/relate to?

NOISSUE

List any changes that modify/break current functionality

None

Have you included tests for your changes?

No

Did you document any new/modified functionality?

No

Notes

N/A

@rodneyosodo rodneyosodo marked this pull request as ready for review March 29, 2023 11:00
@rodneyosodo rodneyosodo requested a review from a team as a code owner March 29, 2023 11:00
@codecov-commenter
Copy link

codecov-commenter commented Mar 29, 2023

Codecov Report

Merging #1754 (70c9a0f) into clients (9abe874) will not change coverage.
The diff coverage is 66.66%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff            @@
##           clients    #1754   +/-   ##
========================================
  Coverage    62.54%   62.54%           
========================================
  Files          122      122           
  Lines         8968     8968           
========================================
  Hits          5609     5609           
  Misses        2792     2792           
  Partials       567      567           
Impacted Files Coverage Δ
pkg/sdk/go/sdk.go 76.47% <ø> (ø)
pkg/sdk/go/things.go 0.00% <0.00%> (ø)
things/policies/page.go 100.00% <ø> (ø)
pkg/sdk/go/users.go 64.56% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>
Signed-off-by: rodneyosodo <blackd0t@protonmail.com>
Signed-off-by: rodneyosodo <blackd0t@protonmail.com>
Signed-off-by: rodneyosodo <blackd0t@protonmail.com>
Signed-off-by: rodneyosodo <blackd0t@protonmail.com>
@dborovcanin dborovcanin merged commit b4eb4a5 into absmach:clients Apr 17, 2023
dborovcanin pushed a commit that referenced this pull request Apr 17, 2023
* Sync Things API Documentation

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Update Secret SDK

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Update Password CLI

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix decodeUpdateClientCredentials Omission

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Failing Update Client Tests

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

---------

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>
dborovcanin pushed a commit that referenced this pull request Apr 17, 2023
* Sync Things API Documentation

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Update Secret SDK

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Update Password CLI

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix decodeUpdateClientCredentials Omission

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Failing Update Client Tests

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

---------

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>
Signed-off-by: dusanb94 <dusan.borovcanin@mainflux.com>
SammyOina pushed a commit to SammyOina/mainflux that referenced this pull request Apr 28, 2023
* Sync Things API Documentation

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Update Secret SDK

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Update Password CLI

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix decodeUpdateClientCredentials Omission

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Failing Update Client Tests

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

---------

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>
Signed-off-by: dusanb94 <dusan.borovcanin@mainflux.com>
SammyOina pushed a commit to SammyOina/mainflux that referenced this pull request Apr 28, 2023
* Sync Things API Documentation

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Update Secret SDK

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Update Password CLI

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix decodeUpdateClientCredentials Omission

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Failing Update Client Tests

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

---------

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>
Signed-off-by: dusanb94 <dusan.borovcanin@mainflux.com>
Signed-off-by: SammyOina <sammyoina@gmail.com>
SammyOina pushed a commit to SammyOina/mainflux that referenced this pull request Apr 28, 2023
* Sync Things API Documentation

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Update Secret SDK

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Update Password CLI

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix decodeUpdateClientCredentials Omission

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

* Fix Failing Update Client Tests

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>

---------

Signed-off-by: rodneyosodo <blackd0t@protonmail.com>
Signed-off-by: dusanb94 <dusan.borovcanin@mainflux.com>
Signed-off-by: SammyOina <sammyoina@gmail.com>
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.

3 participants