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

Incorrect value returned when last task was dropped #835

Closed
rpemberton opened this issue Mar 20, 2023 · 1 comment · Fixed by #839 or #864 · May be fixed by MichalBryxi/ember-resources#1
Closed

Incorrect value returned when last task was dropped #835

rpemberton opened this issue Mar 20, 2023 · 1 comment · Fixed by #839 or #864 · May be fixed by MichalBryxi/ember-resources#1

Comments

@rpemberton
Copy link
Contributor

The value from the dropped task (null) is being returned instead of returning the value from the last non dropped task.

I expect this affects not only dropped tasks but cancelled tasks in general.

PR where bug was introduced: #808
Conversation: #808 (comment)

PR incoming.

@rpemberton rpemberton changed the title Incorrect value returned when last task was droppped Incorrect value returned when last task was dropped Mar 20, 2023
rpemberton added a commit to rpemberton/ember-resources that referenced this issue Mar 20, 2023
Ensure that cancelled tasks are ignored
rpemberton added a commit to rpemberton/ember-resources that referenced this issue Mar 20, 2023
…opped

Ensure that cancelled tasks are ignored
rpemberton added a commit to rpemberton/ember-resources that referenced this issue Mar 20, 2023
…opped

Ensure that cancelled tasks are ignored
rpemberton added a commit to rpemberton/ember-resources that referenced this issue Mar 20, 2023
…opped

Ensure that cancelled tasks are ignored
NullVoxPopuli added a commit that referenced this issue Mar 20, 2023
@NullVoxPopuli
Copy link
Owner

NullVoxPopuli added a commit that referenced this issue Mar 20, 2023
fix: #835 incorrect value returned when last task was dropped
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants