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

Introducing product brand guidelines #138

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

shahmayur001
Copy link

@shahmayur001 shahmayur001 commented Dec 2, 2024

This PR updates the documentation to guide developers on how to customize the brand selection logic in Solidus. It explains how to configure a custom brand selector class, allowing businesses to define brand logic that suits their needs.

The update helps developers understand how to easily adjust the default behavior without altering core code, giving them more flexibility while keeping everything clear and simple.

Checklist

  • I have followed the Diátaxis framework in my PR.
  • I have verified that the preview environment works correctly.

@shahmayur001 shahmayur001 marked this pull request as ready for review December 2, 2024 20:21
@fthobe
Copy link
Contributor

fthobe commented Dec 2, 2024

@kennyadsl can you take a look?

@kennyadsl
Copy link
Member

This looks fantastic! I'll approve once we merge the implementation on Core, thanks @shahmayur001 and @fthobe!

puts product.brand # Outputs the result of your custom logic
```

[order_adjustments_recalculator]: https://github.com/solidusio/solidus/blob/v3.3/core/app/models/spree/promotion/order_adjustments_recalculator.rb
Copy link
Member

Choose a reason for hiding this comment

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

Please remove this link.

Copy link
Contributor

Choose a reason for hiding this comment

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

@kennyadsl we will have to put any other changes or contributions on hold till we understand the governance of the repo better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants