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.

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

[Admin] Add order/show/summary component #5512

Merged
merged 2 commits into from
Nov 17, 2023

Conversation

rainerdema
Copy link
Contributor

@rainerdema rainerdema commented Nov 16, 2023

Summary

Screenshot 2023-11-16 at 11 18 06 Screenshot 2023-11-17 at 12 30 44 Screenshot 2023-11-17 at 12 31 11

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

The following are not always needed:

  • 📖 I have updated the README to account for my changes.
  • 📑 I have documented new code with YARD.
  • 🛣️ I have opened a PR to update the guides.
  • ✅ I have added automated tests to cover my changes.
  • 📸 I have attached screenshots to demo visual changes.

@rainerdema rainerdema self-assigned this Nov 16, 2023
@rainerdema rainerdema force-pushed the rainerd/admin/order-summary branch 2 times, most recently from 7a32cb6 to 82c5c12 Compare November 17, 2023 11:31
This reusable component is designed to display a list of key-value
pairs in a clean and organized format, suitable for various use cases
like order summaries, product specifications, and user profiles.
In this commit, we've added the `order/show/summary` component, 
leveraging
the recently introduced `ui/details_list`.
The component is specifically tailored to display order-related 
information
such as subtotal, taxes, shipping costs, promo code deductions,
adjustments, and the total amount.
@rainerdema rainerdema changed the title [Admin] Add order summary component [Admin] Add order/show/summary component Nov 17, 2023
@rainerdema rainerdema marked this pull request as ready for review November 17, 2023 11:41
@rainerdema rainerdema requested a review from a team as a code owner November 17, 2023 11:41
@rainerdema rainerdema requested a review from elia November 17, 2023 11:41
@rainerdema rainerdema force-pushed the rainerd/admin/order-summary branch from 82c5c12 to c7d6123 Compare November 17, 2023 11:45
Copy link
Member

@elia elia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

render_with_template
end

# @param items select { choices: [[Order details, './data/example1.json'], [Product details, './data/example2.json'], [Account details, './data/example3.json']] }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤩

@rainerdema rainerdema merged commit 9413383 into solidusio:main Nov 17, 2023
2 checks passed
@rainerdema rainerdema deleted the rainerd/admin/order-summary branch November 17, 2023 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants