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

Track webhook intent events as Solidus log entries on the corresponding payment #171

Closed
2 tasks
elia opened this issue Feb 13, 2023 · 3 comments
Closed
2 tasks

Track webhook intent events as Solidus log entries on the corresponding payment #171

elia opened this issue Feb 13, 2023 · 3 comments
Assignees
@elia
Milestone
v5

Comments

@elia
Copy link
Member

elia commented Feb 13, 2023 •
edited
Loading

Why

We want a local copy of the events affecting a given stripe payment, so to aid debugging payments (e.g. customer service trying to shed light on order issues).

What

Add log entries for each payment transition handled by the intents controller.

Dependencies
Preview Give feedback
@elia elia mentioned this issue Feb 13, 2023
Closed
@elia elia changed the title track all the attempts made on stripe as Solidus log entries (check if there's a way to know if a payment update is stale, given they can come also from webhooks and have latency) Track all the attempts made on stripe as Solidus log entries Feb 13, 2023
@elia elia assigned elia and unassigned elia Feb 13, 2023
@loicginoux
Copy link
Contributor

Does this belong to a record in a Spree::LogEntry ? Should we just store it or is there more logic around it ? Should it be displayed somehow in admin ?

@elia
Copy link
Member Author

elia commented Feb 17, 2023

@loicginoux yes, it's related to LogEntry, to which Spree::Payment is connected. In terms of showing the entries they are already visible through the admin in the details page of the payment, although there's surely room for improvement.

@elia elia self-assigned this Feb 22, 2023
@elia elia added this to the v5 milestone Feb 22, 2023
@elia elia changed the title Track all the attempts made on stripe as Solidus log entries Track webhook intent events as Solidus log entries on the corresponding payment Mar 7, 2023
@elia
Copy link
Member Author

elia commented Mar 21, 2023

We agreed to add log entries for all webhooks involving payments from within the individual webhooks issues.

@elia elia closed this as completed Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@elia elia

Labels
None yet
Projects
None yet
Milestone
v5
Development

No branches or pull requests
2 participants
@elia @loicginoux