`ix_image_url` doesn't work with fingerprinted assets anymore about imgix-rails HOT 3 CLOSED

@luqven hi! Thanks for the very quick response, it's very much appreciated 🙏 .

Yes, one easy and quick way for fixing this would be by setting disable_path_encoding, and I think we could benefit from that change. I'll be happy to provide a PR but currently I think I don't have the capacity for doing it, unfortunately I've been rather busy lately... but I'll try to find some time for it in the next days 🤞

from imgix-rails.

Hey @spaghetticode 👋🏼

I just wanted to give you a big shoutout for all your hard work and for giving back to the community. To say thanks, we'd love to hook you up with some account credit. Please get in touch with [email protected] with your account info and send them a link to this comment. They'll get you set up.

We couldn't do it without people like you, and we are grateful for all you do to make imgix-rails the best it can be!

from imgix-rails.

Hey @spaghetticode, I just noticed this was the "parent" issue.

I can see why this is a breaking change for you. We should have more explicitly communicated that in the release notes and I'll make a note to add a test for URLs that include a ? in the path so that any future breaking changes there are caught before release.

Looks like to resolve your issues we'd need to expose a way to set disable_path_encoding globally in imgix-rails. I can't give you a time estimate as to when, but we'll look into adding this as a configuration option.

Of course we're always open to PRs and if you feel comfortable taking a stab we'd love to provide guidance and feedback.

In the meantime if anyone else would like to see this added please do comment and let us know to help us prioritize this work.

Thanks again for reporting this!

from imgix-rails.