WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Nov 18, 2025

There are breaking and it is not worth supporting both
in the same Gem version

@tvdeyen tvdeyen self-assigned this Nov 18, 2025
@tvdeyen tvdeyen changed the title wip: Alchemy 8 Update to Alchemy 8.0 Nov 18, 2025
@tvdeyen tvdeyen added this to the 4.0 milestone Nov 18, 2025
There are breaking and it is not worth supporting both
in the same Gem version
We need an asset pipeline for the dummy app.

Signed-off-by: Thomas von Deyen <[email protected]>
It has changed from positional key and value arguments
to a config hash.

Signed-off-by: Thomas von Deyen <[email protected]>
Alchemy 8.0 disables caching for restricted pages
and now returns a max-age of 0.

Signed-off-by: Thomas von Deyen <[email protected]>
Do not prepend image to file mime type. Alchemy 8 now returns
the proper mime type. Also the picture factory does not allow
anymore to pass the file size. Let’s use the size from the actual
image used in the factory.

Signed-off-by: Thomas von Deyen <[email protected]>
The factories now need to create the attachment.
Stubbing does not work anymore with Alchemy 8 and
its attachment adapters.

Signed-off-by: Thomas von Deyen <[email protected]>
@tvdeyen tvdeyen added breaking-change Changes that need a major release and removed breaking-change Changes that need a major release labels Nov 18, 2025
@tvdeyen tvdeyen marked this pull request as ready for review November 18, 2025 16:31
@tvdeyen tvdeyen merged commit 42bdb53 into main Nov 18, 2025
9 checks passed
@tvdeyen tvdeyen deleted the alchemy-8 branch November 18, 2025 16:59
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.

2 participants