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

Change any link to dispute details link to transaction details link – client. #6929

Closed
Tracked by #6883
shendy-a8c opened this issue Aug 4, 2023 · 1 comment · Fixed by #7087
Closed
Tracked by #6883

Change any link to dispute details link to transaction details link – client. #6929

shendy-a8c opened this issue Aug 4, 2023 · 1 comment · Fixed by #7087
Assignees
@Jinksi
Labels
category: projects For any issues which are part of any project, including bugs, enhancements, etc. component: disputes Issues related to Disputes status: has pr The issue has a PR.

Comments

@shendy-a8c
Copy link
Contributor

shendy-a8c commented Aug 4, 2023 •
edited by haszari
Loading

Since we are showing the dispute details on the transaction details page, any links to the Dispute details page (i.e. this URL/wp-admin/admin.php?page=wc-admin&path=%2Fpayments%2Fdisputes%2Fdetails&id={dispute_id}) now need to link to the relevant changed to Transaction details screen (e.g. /wp-admin/admin.php?page=wc-admin&path=%2Fpayments%2Ftransactions%2Fdetails&id={id}).

The goal of this issue is to update any links to dispute details anywhere in WooPayments admin and ensure they work correctly (i.e. link to relevant transaction).
@shendy-a8c shendy-a8c mentioned this issue Aug 4, 2023
Closed
28 tasks
@shendy-a8c shendy-a8c added component: disputes Issues related to Disputes category: projects For any issues which are part of any project, including bugs, enhancements, etc. labels Aug 4, 2023
@Jinksi Jinksi self-assigned this Aug 29, 2023
This was referenced Aug 29, 2023
Merged
Merged
@Jinksi Jinksi linked a pull request Aug 29, 2023 that will close this issue
Change Dispute → Details links to Transaction → Details #7087
Merged
6 tasks
@Jinksi Jinksi added the status: has pr The issue has a PR. label Aug 29, 2023
@Jinksi Jinksi mentioned this issue Aug 29, 2023
Merged
6 tasks
@Jinksi

This comment was marked as outdated.

Sign in to view
@Jinksi Jinksi removed their assignment Sep 15, 2023
@haszari haszari assigned haszari and Jinksi and unassigned haszari Sep 25, 2023
@Jinksi Jinksi removed their assignment Sep 25, 2023
@Jinksi Jinksi self-assigned this Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Jinksi Jinksi

Labels
category: projects For any issues which are part of any project, including bugs, enhancements, etc. component: disputes Issues related to Disputes status: has pr The issue has a PR.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change Dispute → Details links to Transaction → Details Automattic/woocommerce-payments
3 participants
@Jinksi @haszari @shendy-a8c