-
Notifications
You must be signed in to change notification settings - Fork 22
Users/brian poteet ni/export openapi for 21 3 #95
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
Open
brian-poteet-ni
wants to merge
35
commits into
master
Choose a base branch
from
users/brian-poteet-ni/export-openapi-for-21-3
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
009a374
Add BasicAuth to all without it
juno-jin-ni 7a8e2b1
Merge pull request #91 from ni/allow-basic-auth
juno-jin-ni 9373ed3
Update SystemMetadata
e8d5b33
Merge pull request #92 from ipopa144/users/ipopa/update-systemmetadata
76e0dac
Merge pull request #94 from ni/users/vladbaja/export-openapi-for-21-1
vladbaja a5005d1
Merged PR 143611: Added check for all system properties updated
vdeac 50bcff9
Merged PR 143598: Pull OpenAPI docs from GitHub subrepo
176d380
Merged PR 147691: Extend FileIngestion Service to allow force delete …
Everest2020 2a7164c
Merged PR 145036: This is a proposed fix for the bug 1355793 allowing…
fabcd1f
Merged PR 147792: Add missing Swagger documentation for tag historian
7137392
Merged PR 153636: Add route to link files to an asset
vladbaja 7e52cae
Merged PR 154444: Add fileIds property on yml
4716a54
Merged PR 154086: Updated the AssetUpdateModel in YML to include REMO…
tludusan 40da1be
Merged PR 157149: Export notebooks as HTML or PDF
aa5fba0
Merged PR 158162: US 1449042: Support workspace filtering in Query Files
f8cfd7e
Merged PR 164847: Add `unlink file` route to swagger
826a95e
Merged PR 165886: Bug 1475922 - Updated swagger docs with recent work…
lnowotny 1a89269
Merged PR 169224: Move values field under properties
brian-poteet-ni a6b5135
Merged PR 170365: Fix description in nifile.yml
ab7f895
Merged PR 172517: Increase delete-files operation version
a49541f
Add example on how to filter systems by DateTime values inside dictio…
ArinLucaNI 8f9fa57
Merged PR 178881: Updated swagger doc for asset summary route
vdeac fbe0a1e
Merged PR 179041: [BuildingFinals] Pull systemlink-openapi-documents …
vladbaja 39c82d6
Merged PR 179689: Use valid query-steps filter in swagger example
mollykreis 14c1ea5
Merged PR 186899: Updated yml with SupportsSelfTest and SupportsReset
tludusan 4afea20
Merged PR 186908: Update link for Dynamic LINQ
chotea ace1daa
Merged PR 188023: Fix build that auto-generates python clients from s…
andzn 43a763e
Merged PR 188055: Fix YAML format
5348eac
Merged PR 194959: Make corrections to test monitor swagger documentation
siddhukrs d29f6c9
Merged PR 195386: Revert PR 194959: Changes I made to the swagger doc…
siddhukrs 8423020
Merged PR 195476: Attempt #2: Make corrections to TestMonitor's swagg…
siddhukrs bd306a0
Merged PR 200198: Fix docs
haworthia 792fe2c
Merged PR 207128: Add lastUpdatedTimestamp field to file model
codrutagherman 08ab170
Merged PR 208425: Update TestMonitor API docs to add required tags fo…
adamarnesen a1373b3
Merged PR 216118: Pull systemlink-openapi-documents from GitHub
brian-poteet-ni File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -12,8 +12,11 @@ securityDefinitions: | |
| type: apiKey | ||
| name: x-ni-api-key | ||
| in: header | ||
| BasicAuth: | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same here: BasicAuth is not supported |
||
| type: basic | ||
| security: | ||
| - ApiKeyAuth: [] | ||
| - BasicAuth: [] | ||
| paths: | ||
| '/users/query': | ||
| post: | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -12,8 +12,11 @@ securityDefinitions: | |
| type: apiKey | ||
| name: x-ni-api-key | ||
| in: header | ||
| BasicAuth: | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this doesn't look right. no new APIs support BasicAuth |
||
| type: basic | ||
| security: | ||
| - ApiKeyAuth: [] | ||
| - BasicAuth: [] | ||
| paths: | ||
| '/webapps/{id}': | ||
| get: | ||
|
|
||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
we will no longer support basic auth