Skip to content

Update: Use Ember concurrency for data interfaces/adapters#1333

Merged
kevinhinterlong merged 5 commits intomasterfrom
ember-concurrency-data
Mar 29, 2021
Merged

Update: Use Ember concurrency for data interfaces/adapters#1333
kevinhinterlong merged 5 commits intomasterfrom
ember-concurrency-data

Conversation

@kevinhinterlong
Copy link
Member

Description

Update facts/dimension adapters and services to use ember-concurrency tasks. Cancels elide queries we no longer use

Proposed Changes

  • Convert most data interfaces/adapters to be task based
  • dimension select local search if all results are fetched
  • update typescript/ember-concurrency

License

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@kevinhinterlong kevinhinterlong force-pushed the ember-concurrency-data branch 2 times, most recently from 5124f39 to e3efa40 Compare March 23, 2021 21:19
Copy link
Contributor

@jkusa jkusa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have any tests that exercise the cancel functionality?

@kevinhinterlong kevinhinterlong force-pushed the ember-concurrency-data branch from 4c18f87 to 1b3933b Compare March 29, 2021 18:32
@kevinhinterlong kevinhinterlong merged commit e9fd291 into master Mar 29, 2021
@kevinhinterlong kevinhinterlong deleted the ember-concurrency-data branch March 29, 2021 19:05
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.

2 participants