-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Feature/env error controls #5216
Closed
eic-chr
wants to merge
35
commits into
swagger-api:master
from
HUK-COBURG:feature/env-error-controls
Closed
Feature/env error controls #5216
eic-chr
wants to merge
35
commits into
swagger-api:master
from
HUK-COBURG:feature/env-error-controls
Conversation
This file contains 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
environment variable
feat: Do spectral validation calls
Allows switching between different rulesets, might be useful later on. Currently supported values: v5,v10
…n-for-spectral-rules
Snyk has created this PR to upgrade react-redux from 7.2.8 to 7.2.9. See this package in npm: https://www.npmjs.com/package/react-redux See this project in Snyk: https://app.snyk.io/org/huk-coburg/project/6781dd8e-dbe8-4e84-86b5-f2f4515c956a?utm_source=github&utm_medium=referral&page=upgrade-pr
…63d5639f754488 [Snyk] Upgrade react-redux from 7.2.8 to 7.2.9
…al-rules First draft: Dropdown for spectral rules
Snyk has created this PR to upgrade swagger-client from 3.19.2 to 3.19.3. See this package in npm: https://www.npmjs.com/package/swagger-client See this project in Snyk: https://app.snyk.io/org/huk-coburg/project/6781dd8e-dbe8-4e84-86b5-f2f4515c956a?utm_source=github&utm_medium=referral&page=upgrade-pr
Backport of changes introduced in #6. Not entirely sure what happens here, this seems to be a timing issue. Not giving an explicit `source` to the messages means they go into the same pool as the default messages, this also means that they fall into the same cleanup logic. The `newSpecErrBatch` function seems to mess with that cleanup (our spectral messages clean up the "regular messages") and the UI tries to render messages that no longer exist, crashing in the process. the `newThrownErrBatch` function doesn't seem to have these issues but it requires a debounce for some reason or it won't render anything.
…fined fix(spectral): Fix validation crashing the UI
We might have in-flight validation attempts that need to be cancelled when a new one is about to be issued or we risk getting the same issue displayed multiple times if the timing is exactly right/wrong and both complete after all `arg.errActions.clear` have run. To work around this we use an AbortController, fetch has built-in support for that. Sadly the easy solution of globally creating a controller and then simply sending an abort() on every run of validateSpec will immediately cancel the new fetch that is supposed to happen. To work around this a new AbortController is constructed on every run of validateSpec and stored in the controller variable. The controller variable is null at start and will be set back to null after every fetch() call. If it isn't null when validateSpec is called then this must mean that a request is currently going on and that can be cancelled. Overriding the controller afterwards with a new AbortController will avoid the signal also cancelling the new fetch() call Co-authored-by: Erlwein, Björn <[email protected]>
…4525545682bac0 [Snyk] Upgrade swagger-client from 3.19.2 to 3.19.3
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Checklist
My PR contains...
src/
is unmodified: changes to documentation, CI, metadata, etc.)package.json
)My changes...
Documentation
Automated tests