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

expose translations in global Alchemy js object, #2113 #2114

Merged
merged 1 commit into from
Jun 4, 2021
Merged

expose translations in global Alchemy js object, #2113 #2114

merged 1 commit into from
Jun 4, 2021

Conversation

afdev82
Copy link
Contributor

@afdev82 afdev82 commented Jun 4, 2021

What is this pull request for?

Closes #2113 and it's a prerequisite for AlchemyCMS/alchemy_i18n#34
It needs to be backported to all v5.x versions of Alchemy, because since Webpacker was added, the gem is broken.

Notable changes (remove if none)

The global JS object Alchemy now contains the key translations like in Alchemy v4, in order to use it to add other translations using the alchemy_i18n gem.

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Ah. Now I get the problem. Yes, this is indeed a bug we need to backport to v5 branches. Thanks for fixing.

@tvdeyen tvdeyen merged commit c03a531 into AlchemyCMS:main Jun 4, 2021
@tvdeyen
Copy link
Member

tvdeyen commented Jun 7, 2021

Dang this never passed Jest specs :(

tvdeyen added a commit to tvdeyen/alchemy_cms that referenced this pull request Jun 7, 2021
They were missing from AlchemyCMS#2114
@tvdeyen tvdeyen mentioned this pull request Jun 7, 2021
tvdeyen added a commit that referenced this pull request Jun 8, 2021
tvdeyen added a commit that referenced this pull request Jun 8, 2021
robinboening pushed a commit to robinboening/alchemy_cms that referenced this pull request Jul 2, 2021
@afdev82 afdev82 deleted the fix_translations branch July 8, 2021 14:51
tvdeyen added a commit that referenced this pull request Sep 15, 2021
- Backport #2115 to v5.0 [#2126](#2126) ([dbwinger](https://github.com/dbwinger))
- Fixate Dragonfly to < 1.4 [#2122](#2122) ([tvdeyen](https://github.com/tvdeyen))
- Backport #2114 to v5.0 [#2118](#2118) ([afdev82](https://github.com/afdev82))
tvdeyen added a commit that referenced this pull request Sep 15, 2021
- generate picture thumbnails only for pictures with convertible format [#2131](#2131) ([afdev82](https://github.com/afdev82))
- Backport #2115 to v5.1 [#2125](#2125) ([dbwinger](https://github.com/dbwinger))
- Backport #2114 to v5.1 [#2117](#2117) ([afdev82](https://github.com/afdev82))
tvdeyen added a commit that referenced this pull request Sep 15, 2021
- Return only pages from current site in api [#2169](#2169) ([afdev82](https://github.com/afdev82))
- Improve cache key defaults for menus #2138 [#2160](#2160) ([oneiros](https://github.com/oneiros))
- generate picture thumbnails only for pictures with convertible format [#2130](#2130) ([afdev82](https://github.com/afdev82))
- Backport #2114 to v5.2 [#2116](#2116) ([afdev82](https://github.com/afdev82))
- Add webpacker tasks to Alchemy upgrader [#2115](#2115) ([dbwinger](https://github.com/dbwinger))
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.

Alchemy i18n gem doesn't work with Alchemy v5
2 participants