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

wth - (SPARCRequest) Epic Push Origin Bug #1158

Merged
merged 1 commit into from Oct 17, 2017

Conversation

wtholt
Copy link
Contributor

@wtholt wtholt commented Oct 17, 2017

1). There are only 2 types/origins of Epic push: Submission Push and Admin Push. However, there is a 3rd origin "pi_email_approval" showing up in the epic_queue_records table (see screenshot #1). If those are actually overlord pushes, please use "overlord_push" for data records instead. Please look into what's causing that, make historical data consistent, and fix the bug.

2). In the epic_queues table, it appears to be showing protocols that has already been pushed to Epic too, when it should only show the ones that still awaiting to be pushed. See screenshot #2.

[#151955764]

Story - https://www.pivotaltracker.com/story/show/151955764

1). There are only 2 types/origins of Epic push: Submission Push and Admin Push. However, there is a 3rd origin "pi_email_approval" showing up in the epic_queue_records table (see screenshot #1). If those are actually overlord pushes, please use "overlord_push" for data records instead. Please look into what's causing that, make historical data consistent, and fix the bug.

2). In the epic_queues table, it appears to be showing protocols that has already been pushed to Epic too, when it should only show the ones that still awaiting to be pushed. See screenshot #2.

[#151955764]

Story - https://www.pivotaltracker.com/story/show/151955764
@Stuart-Johnson Stuart-Johnson merged commit 243a054 into v3.0.0b Oct 17, 2017
@Stuart-Johnson Stuart-Johnson deleted the wth-epic-origin-bug branch October 17, 2017 19:38
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