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

Fixed JobTarget OpenAPI definition #3585

Merged
merged 1 commit into from
Aug 23, 2022

Conversation

Coduz
Copy link
Contributor

@Coduz Coduz commented Aug 11, 2022

This PR fixes the definition of JobTarget in Open API.

  • Fixed properties
  • Added missing JobTargetStaus

Related Issue
None

Description of the solution adopted
Fixed the documentation

Screenshots
None

Any side note on the changes made
None

Signed-off-by: Alberto Codutti <alberto.codutti@eurotech.com>
@Coduz Coduz added REST API We hope to REST everyday Documentation Doc... What?? Joke! We will write some! labels Aug 11, 2022
@Coduz Coduz requested a review from riccardomodanese August 11, 2022 13:46
@codecov
Copy link

codecov bot commented Aug 11, 2022

Codecov Report

Merging #3585 (d0ea8a7) into develop (423aaa8) will decrease coverage by 35.08%.
The diff coverage is n/a.

❗ Current head d0ea8a7 differs from pull request most recent head 8f051b1. Consider uploading reports for the commit 8f051b1 to get more accurate results

Impacted file tree graph

@@              Coverage Diff               @@
##             develop    #3585       +/-   ##
==============================================
- Coverage      50.20%   15.12%   -35.09%     
+ Complexity       259       72      -187     
==============================================
  Files           1662     1662               
  Lines          31946    31946               
  Branches        2643     2643               
==============================================
- Hits           16039     4832    -11207     
- Misses         14970    26679    +11709     
+ Partials         937      435      -502     
Impacted Files Coverage Δ
...rc/main/java/org/eclipse/kapua/KapuaException.java 0.00% <0.00%> (-100.00%) ⬇️
...c/main/java/org/eclipse/kapua/KapuaErrorCodes.java 0.00% <0.00%> (-100.00%) ⬇️
...in/java/org/eclipse/kapua/broker/BrokerDomain.java 0.00% <0.00%> (-100.00%) ⬇️
...java/org/eclipse/kapua/commons/util/ClassUtil.java 0.00% <0.00%> (-100.00%) ⬇️
...ain/java/org/eclipse/kapua/event/ServiceEvent.java 0.00% <0.00%> (-100.00%) ⬇️
...n/java/org/eclipse/kapua/broker/BrokerDomains.java 0.00% <0.00%> (-100.00%) ⬇️
...va/org/eclipse/kapua/commons/util/SystemUtils.java 0.00% <0.00%> (-100.00%) ⬇️
...n/java/org/eclipse/kapua/model/domain/Actions.java 0.00% <0.00%> (-100.00%) ⬇️
.../java/org/eclipse/kapua/message/KapuaPosition.java 0.00% <0.00%> (-100.00%) ⬇️
.../java/org/eclipse/kapua/KapuaRuntimeException.java 0.00% <0.00%> (-100.00%) ⬇️
... and 718 more

@Coduz Coduz merged commit c08303a into eclipse:develop Aug 23, 2022
@Coduz Coduz deleted the fix-JobTargetStatusOpenApi branch August 23, 2022 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Doc... What?? Joke! We will write some! REST API We hope to REST everyday
Projects
Development

Successfully merging this pull request may close these issues.

2 participants