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.

Sign up for GitHub

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

Jump to bottom

Split out execute errors that come from authenticated calls #3589

Open
3 tasks
just-a-node opened this issue Feb 23, 2023 · 6 comments
Open
3 tasks

Split out execute errors that come from authenticated calls #3589

just-a-node opened this issue Feb 23, 2023 · 6 comments
Labels
Enhance 🧘 Issue type: Enhancement Request integrations

Comments

@just-a-node
Copy link
Collaborator

just-a-node commented Feb 23, 2023 •
edited
Loading

Problem

Auth check calls will fall under the generic Execution Failure and this will bubble up to the UI. Ideal case is all integrators use a standard onlySource modifier that fails with a standard error message. Then we can parse out this message and show authenticated executes as "Pending Authentication" (or something) in the UI.

Proposed Solution
@just-a-node just-a-node added Enhance 🧘 Issue type: Enhancement Request integrations labels Feb 23, 2023
@alexwhte alexwhte added the P2 label Mar 10, 2023
@alexwhte alexwhte removed the P2 label Mar 30, 2023
@just-a-node just-a-node added the Pri label Apr 6, 2023
@rhlsthrm
Copy link
Collaborator

rhlsthrm commented Apr 6, 2023

Complexity: 13
See the discussion in Sprint Poker #​​26

Powered by Parabol

@rhlsthrm
Copy link
Collaborator

rhlsthrm commented Apr 6, 2023

Impact: 4
See the discussion in Sprint Poker #​​27

Powered by Parabol

@rhlsthrm
Copy link
Collaborator

rhlsthrm commented Apr 6, 2023

Urgency: 3
See the discussion in Sprint Poker #​​27

Powered by Parabol

@rhlsthrm rhlsthrm removed the Pri label Apr 12, 2023
@rhlsthrm
Copy link
Collaborator

Complexity: 13
See the discussion in Sprint Poker #​​29

Powered by Parabol

@rhlsthrm
Copy link
Collaborator

Impact: 3
See the discussion in Sprint Poker #​​30

Powered by Parabol

@rhlsthrm
Copy link
Collaborator

Urgency: 3
See the discussion in Sprint Poker #​​30

Powered by Parabol

@alexwhte alexwhte removed the Pri label Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhance 🧘 Issue type: Enhancement Request integrations
Projects
Priority Rank
Status: No status
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rhlsthrm @just-a-node @alexwhte