feat: enable redirection button to order or enrollment request #20
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@andres-arango-aptitude, can you help us with your review? |
fix: phpcs spacing in comments fix: phpcs spacing fixes fix: spaces instead of tabs fix: phpcs spacing and characters control fix: echo line exceeds 85 characters fix: phpcs fixes in admin file fix: general phpcs fixes in info form file fix: phpcs fixes in main plugi file fix: character limit reached fix: characters limit reached in main file fix: phpcs spaces fixed fix: characters limit reached in main file fix: plugin file spacing fixed fix: log manager variable name fixed
MaferMazu
force-pushed
the
enrollment-order-redirect
branch
from
2a8ffd7 to
ac3fcd6
Compare
MaferMazu
reviewed
Aug 10, 2023
|
Can you add screenshots to the PR and briefly explain how this feature works? |
|
I have included the screenshots in the "Additional Information" section. |
MaferMazu
approved these changes
Aug 15, 2023
I prefer to fix it in the visual ergonomy pull requests to leave this PR clean as possible. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
The functionality has been added to the Enrollment Request, so that whenever there is an associated order_id, a button labeled "View Order" is enabled, allowing users to be redirected to the associated order with just one click. Now, within an order, in the product items, if an Enrollment Request ID is set for them, a button is enabled that allows users to be redirected to the Enrollment Request associated with that item in that order.
Testing instructions
Go to the Enrollment Request and enter an order_id in the order_id field, then save the changes. Click on the button to be redirected to the associated order. Similarly, from an order, if any product item has a set Enrollment Request ID, click on the corresponding button to be redirected to the linked Enrollment Request.
Additional information
To show it better I will explain the functionalities with pictures:
If you input the order number in the Enrollment Request, the "View Order" button will become active.
When you access the order, you will come across a field that contains items classified as courses.
If a field is empty, it will appear disabled.
And also like this in order items.
Checklist for Merge