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

Update the order links when BuddyPress is active. #2432

Open
dominiquemariano opened this issue May 11, 2023 · 1 comment · May be fixed by #2466
Open

Update the order links when BuddyPress is active. #2432

dominiquemariano opened this issue May 11, 2023 · 1 comment · May be fixed by #2466
Assignees
@actuallyakash
Labels
Reviewed Severity: Low Status: Has Alternative A workaround or alternative solution is available Type: Enhancement Improvements existing features or code

Comments

@dominiquemariano
Copy link

Is your feature request related to a problem? Please describe.
With BuddyPress and LifteLMS active, when you visit /members/admin/courses/orders/ and click on any order link (including the "View" button), it redirects you to the /dashboard/orders/<order number> instead of /members/admin/courses/<order number>. You can watch this video (sent to the customer) to see what I mean.

Describe the solution you'd like
When you visit /members/admin/courses/orders/ and click on any order link (including the "View" button), it should redirect you to /members/admin/courses/<order number> instead of /dashboard/orders/<order number>.

Describe alternatives you've considered
You can override this with a child theme, but it's better to fix this from within LifterLMS.

Side effects
This should also fix the problem with BuddyBoss.

Affected tickets
Affected customers include HS-216423.
@eri-trabiccolo eri-trabiccolo added Type: Enhancement Improvements existing features or code Severity: Low Status: Has Alternative A workaround or alternative solution is available labels May 11, 2023
@actuallyakash actuallyakash self-assigned this Jun 22, 2023
@actuallyakash actuallyakash linked a pull request Jun 22, 2023 that will close this issue
Draft
3 tasks
@actuallyakash actuallyakash added the Status: Has PR This issue has an open pull request label Jun 22, 2023
@eri-trabiccolo eri-trabiccolo removed the Status: Has PR This issue has an open pull request label Jul 19, 2023
@brianhogg
Copy link
Contributor

Worth doing if we can find a secure way to do it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@actuallyakash actuallyakash

Labels
Reviewed Severity: Low Status: Has Alternative A workaround or alternative solution is available Type: Enhancement Improvements existing features or code
Projects
Development
Status: Backlog
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue/2432 actuallyakash/lifterlms
4 participants
@brianhogg @eri-trabiccolo @actuallyakash @dominiquemariano