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

Translatable Fields #145

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jarednorman
Copy link
Member

Some stores may not want to translate all fields, or may have additional fields that they want to translate. This allows stores to configure that themselves.

Graham Bouvier and others added 4 commits September 16, 2021 16:38
Previously, translatable fields were hard-coded in the various decorators.
With this change, they can be overridden with configuration in the client application
using similar configuration patterns to other solidus extensions.
This was blocking support for Ruby 3.0 in this extension.
@stale
Copy link

stale bot commented Sep 21, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Sep 21, 2022
@jarednorman
Copy link
Member Author

@omnistegan I don't know if you're interested, but I would love to merge this PR, but the builds on this PR are borked... any chance you have a second to look at that?

@stale stale bot removed the wontfix label Sep 21, 2022
@stale
Copy link

stale bot commented Jan 7, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Jan 7, 2023
@jarednorman jarednorman added pinned and removed wontfix labels Jan 8, 2023
@jarednorman
Copy link
Member Author

We're definitely going to want to merge this eventually once it gets fixed up. We're currently using it in production.

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

Successfully merging this pull request may close these issues.

1 participant