-
Notifications
You must be signed in to change notification settings - Fork 58
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
feat: improved data fetching for execution details page #131
Merged
Commits on Jan 5, 2021
-
refactor: integrate react-query for execution data fetching (#123)
* refactor: using react-query to load top level Execution * refactor: upgrading react-query and fixing execution termination * refactor: handle 401s on queries and do auth flow * refactor: adding conditional refresh for execution status * refactor: cleanup broken files after context refactor * chore: docs
Configuration menu - View commit details
-
Copy full SHA for 09eb922 - Browse repository at this point
Copy the full SHA 09eb922View commit details -
refactor: Remove ExecutionDataCache in favor of react-query (#126)
* refactor: first step of using queries for NE table * refactor: removing data cache from first layer of NE table * refactor: removing remaining execution data cache usage * refactor: rename QueryKey type and remove bug workaround * refactor: fixing remaining consumers of NEs
Configuration menu - View commit details
-
Copy full SHA for 705303e - Browse repository at this point
Copy the full SHA 705303eView commit details -
test: adds setup for mock-service-worker (#127)
* test: add msw and basic handlers for a few types * test: add mock data for a basic workflow execution * test: fixing/removing tests after adding msw * test: throw on unexpected requests to msw * fix: upgrade TS to fix error and cleanup resulting errors
Configuration menu - View commit details
-
Copy full SHA for b881bcf - Browse repository at this point
Copy the full SHA b881bcfView commit details -
Migrate from TSLint to ESLint (#128)
* ci: move from tslint->eslint * fix: addressing eslint errors * fix: remove passing of unused variable * ci: remove unnecessary prettier config
Configuration menu - View commit details
-
Copy full SHA for 3f0427e - Browse repository at this point
Copy the full SHA 3f0427eView commit details -
refactor: clean up mock fixtures and re-enable tests for executions (#…
…130) * test: adding test data for node executions * test: mocks and refactoring to re-enable NodeExecutionDetails tests * chore: lint error * test: getting first test for NE table working again * test: mocks and a couple of tests for NE table * refactor: msw handlers to use a backing map and return 404s * test: more tests for NE table * test: adding fixture for dynamic external workflow * test: using mock fixture for sub workflow tests * test: move remaining mocks to fixtures and fix tests * test: re-enabling more execution tests * fix: removing global query handlers for caching entitiesq * test: re-enable ExecutionNodeViews tests * fix: typo in import path * fix: show DataError by default for failed queries * chore: documentation * chore: pr feedback
Configuration menu - View commit details
-
Copy full SHA for ee998d4 - Browse repository at this point
Copy the full SHA ee998d4View commit details -
Configuration menu - View commit details
-
Copy full SHA for c99d1cb - Browse repository at this point
Copy the full SHA c99d1cbView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8146296 - Browse repository at this point
Copy the full SHA 8146296View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5ff0502 - Browse repository at this point
Copy the full SHA 5ff0502View commit details -
Configuration menu - View commit details
-
Copy full SHA for 190f258 - Browse repository at this point
Copy the full SHA 190f258View commit details -
Configuration menu - View commit details
-
Copy full SHA for 85440e8 - Browse repository at this point
Copy the full SHA 85440e8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2dd7710 - Browse repository at this point
Copy the full SHA 2dd7710View commit details -
Configuration menu - View commit details
-
Copy full SHA for 84c292c - Browse repository at this point
Copy the full SHA 84c292cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 52c00ed - Browse repository at this point
Copy the full SHA 52c00edView commit details
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.