-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add Order#use_shipping
and address management to the admin dashboard
#5461
Merged
rainerdema
merged 8 commits into
solidusio:main
from
nebulab:rainerd/admin/order/address-form
Nov 8, 2023
Merged
Add Order#use_shipping
and address management to the admin dashboard
#5461
rainerdema
merged 8 commits into
solidusio:main
from
nebulab:rainerd/admin/order/address-form
Nov 8, 2023
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
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
2 times, most recently
from
October 30, 2023 18:49
052c89a
to
788a05a
Compare
rainerdema
changed the title
[Admin] Add order address/customer form
[Admin] Use address form component for order/customer addresses
Oct 30, 2023
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
2 times, most recently
from
November 3, 2023 15:24
b4fa09b
to
ba1f9e1
Compare
rainerdema
changed the base branch from
main
to
elia/admin/orders-customer-sidebar
November 3, 2023 15:24
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
from
November 3, 2023 17:51
ba1f9e1
to
d2380c7
Compare
github-actions
bot
added
changelog:solidus_api
Changes to the solidus_api gem
changelog:solidus_core
Changes to the solidus_core gem
labels
Nov 3, 2023
rainerdema
changed the title
[Admin] Use address form component for order/customer addresses
[Admin] Order process: Integrate address form component for billing and shipping
Nov 3, 2023
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
from
November 3, 2023 18:03
d2380c7
to
dcb4daf
Compare
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
2 times, most recently
from
November 3, 2023 22:31
de15a9d
to
f9c39e2
Compare
elia
force-pushed
the
elia/admin/orders-customer-sidebar
branch
from
November 6, 2023 09:19
77b9f54
to
4a13a62
Compare
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
from
November 6, 2023 09:22
f9c39e2
to
50d922d
Compare
elia
force-pushed
the
elia/admin/orders-customer-sidebar
branch
3 times, most recently
from
November 6, 2023 14:26
7a4861a
to
0dda932
Compare
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
from
November 6, 2023 14:36
50d922d
to
7c80411
Compare
elia
reviewed
Nov 6, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a few comments, but can't wait to merge this! 👏
admin/app/components/solidus_admin/orders/address/component.html.erb
Outdated
Show resolved
Hide resolved
admin/app/components/solidus_admin/orders/address/component.html.erb
Outdated
Show resolved
Hide resolved
admin/app/components/solidus_admin/orders/address/component.yml
Outdated
Show resolved
Hide resolved
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
from
November 7, 2023 13:39
7c80411
to
6b220de
Compare
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
3 times, most recently
from
November 7, 2023 14:10
52d356e
to
fe908f9
Compare
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
2 times, most recently
from
November 7, 2023 15:46
b86d6ff
to
f6e2e26
Compare
Add use_shipping attribute with syncing logic to `Spree::Order` model. Similar to the existing `use_billing` feature, a `use_shipping` attribute has been introduced. This addition enables the automatic assignment of the shipping address to the billing address when the `use_shipping` flag is set. The `before_validation` callbacks ensure that the addresses are synchronized, thereby streamlining the address management process in the new admin interface.
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
from
November 8, 2023 09:39
f6e2e26
to
114759f
Compare
3 tasks
elia
reviewed
Nov 8, 2023
- Introduce a new `AddressesController` to handle the creation and updating of billing and shipping addresses for orders. Includes new and update actions with strong parameter support and address type validation. - Add links for address management to customer section component.
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
from
November 8, 2023 17:49
114759f
to
a2a2b90
Compare
Leverage the existing `ui/forms/address` component to render the address forms within the admin order process.
Modify the style definitions for city and zipcode fields to ensure consistent height alignment with other fields in the form.
Enhance the address form's state field by ensuring the correct state is selected during asynchronous loading. This adjustment addresses a timing issue where Stimulus did not populate the state select promptly, leading to mismatched selections.
Modify the address form logic to accommodate countries without associated states. The state select field is now automatically disabled when a country with no states is selected. Additionally, this ensure state select field is disabled for preloaded selected country without states.
Corrected the Stimulus controller event handling to ensure dropdown <details> close on modal open/close. Previously, the `turbo:before-cache` event was not properly connected to the Stimulus controller, causing menus to remain open after a modal was triggered. The 'closeMenus' method within the controller was attempting to call 'querySelectorAll' on 'this.event', which is undefined within the Stimulus context. This update corrects the 'closeMenus' method to reference 'this.element', ensuring that it correctly targets the controller's DOM element.
rainerdema
force-pushed
the
rainerd/admin/order/address-form
branch
from
November 8, 2023 18:01
a2a2b90
to
3a5daf8
Compare
elia
approved these changes
Nov 8, 2023
elia
changed the title
[Admin] Order process: Integrate address form component for billing and shipping
Add Dec 21, 2023
Order#use_shipping
and address management to the admin dashboard
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
changelog:solidus_admin
changelog:solidus_api
Changes to the solidus_api gem
changelog:solidus_core
Changes to the solidus_core gem
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.
Summary
Integrate.address.form.component.for.billing.and.shipping.mov
Checklist
Check out our PR guidelines for more details.
The following are mandatory for all PRs:
The following are not always needed: