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

feat: error handling/data-module core #14

Merged
merged 1 commit into from
Nov 29, 2021
Merged

feat: error handling/data-module core #14

merged 1 commit into from
Nov 29, 2021

Conversation

diehbria
Copy link
Contributor

  • Prevent data streams with associated errors from re-requesting over
    and over
  • Refactor subscription store to utilize RXJS to broadcast subscription
    updates to subscribers
  • Update types on @iot-app-kit/components to be any query
  • Remove unused dependency on redux-thunk
  • Up required code coverage targets
  • run linting on test
  • add subscribeToDataStreamsFrom stubs

 * Prevent data streams with associated errors from re-requesting over
  and over
 * Refactor subscription store to utilize RXJS to broadcast subscription
  updates to subscribers
 * Update types on @iot-app-kit/components to be any query
 * Remove unused dependency on redux-thunk
 * Up required code coverage targets
 * run linting on test
 * add subscribeToDataStreamsFrom stubs
@diehbria diehbria merged commit cb85241 into main Nov 29, 2021
@diehbria diehbria deleted the core branch November 29, 2021 19:00
sheilaXu added a commit that referenced this pull request Sep 23, 2022
Co-authored-by: Xinyi Xu <xinyxu@amazon.com>
TheEvilDev pushed a commit that referenced this pull request Nov 2, 2022
* Prevent data streams with associated errors from re-requesting over
  and over
 * Refactor subscription store to utilize RXJS to broadcast subscription
  updates to subscribers
 * Update types on @iot-app-kit/components to be any query
 * Remove unused dependency on redux-thunk
 * Up required code coverage targets
 * run linting on test
 * add subscribeToDataStreamsFrom stubs
This was referenced Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants