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

Fix starting multiple async transitions through API #2753

Merged
merged 2 commits into from
Aug 23, 2016

Conversation

mkurek
Copy link
Contributor

@mkurek mkurek commented Aug 23, 2016

Only one active transition could be started at a time. Continuation of #2644.

Only one active transition could be started at a time. Continuation of allegro#2644.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 86.091% when pulling 5e2adcc on mkurek:fix-api-multiple-async-transitions-2 into c171372 on allegro:ng.

@szok
Copy link
Contributor

szok commented Aug 23, 2016

🆗

self.assertEqual(response.status_code, 400)
self.assertEqual(response.data, {
'non_field_errors': [
'Another async transition for this object is already stared'
Copy link

Choose a reason for hiding this comment

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

typo?

@ghost
Copy link

ghost commented Aug 23, 2016

🆗

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 86.02% when pulling 75c6fda on mkurek:fix-api-multiple-async-transitions-2 into c171372 on allegro:ng.

@mkurek mkurek merged commit d5c730e into allegro:ng Aug 23, 2016
@mkurek mkurek removed the review label Aug 23, 2016
ar4s pushed a commit to ar4s/ralph that referenced this pull request Aug 28, 2017
Fix starting multiple async transitions through API

Only one active transition could be started at a time. Continuation of allegro#2644.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants