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

Update API documentation #1187

Merged
merged 1 commit into from
Sep 9, 2022
Merged

Update API documentation #1187

merged 1 commit into from
Sep 9, 2022

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Sep 9, 2022

Automated changes to the Open API documentation.
This PR is related to the change **Define EdgeHub routes in edge models (#1173)

  • Add IoTEdgeRoute Model

  • Add Routes on CreateEdgeModelsPage

  • Add Route inthe EdgeModelMapper

  • Get/Set routes in EdgeHub DesiredProperties

  • Fixing existing tests

  • Add validation on routes + Route is now case insensitive

  • Add tests on IoTEdgeRouteValidator

  • Add tests on ConfigHelper

  • Fix unnecessary using directive

  • Add test on front pages

  • Fix typo on unit test

  • Fix error in EdgeModelDetailPageTest

  • Add test on EdgeModelDetailPage

  • Add test on ConfigService + fix typo** merged in b1d1971.

@github-actions github-actions bot requested a review from a team as a code owner September 9, 2022 15:09
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Sep 9, 2022
@codecov
Copy link

codecov bot commented Sep 9, 2022

Codecov Report

❗ No coverage uploaded for pull request base (docs/main@f75f6ba). Click here to learn what that means.
Patch has no changes to coverable lines.

❗ Current head b1d1971 differs from pull request most recent head 9849f74. Consider uploading reports for the commit 9849f74 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             docs/main    #1187   +/-   ##
============================================
  Coverage             ?   87.92%           
============================================
  Files                ?      203           
  Lines                ?     6891           
  Branches             ?        0           
============================================
  Hits                 ?     6059           
  Misses               ?      832           
  Partials             ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@hocinehacherouf hocinehacherouf enabled auto-merge (squash) September 9, 2022 15:12
@kbeaugrand kbeaugrand disabled auto-merge September 9, 2022 16:46
@kbeaugrand kbeaugrand merged commit 18d4aa3 into docs/main Sep 9, 2022
@kbeaugrand kbeaugrand deleted the docs/feature/open-api branch September 9, 2022 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants