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

Recent getFallbackProxy changes breaks semver versioning #157

Closed
jbergstroem opened this issue Jan 24, 2022 · 4 comments
Closed

Recent getFallbackProxy changes breaks semver versioning #157

jbergstroem opened this issue Jan 24, 2022 · 4 comments
Labels
bug Something isn't working

Comments

@jbergstroem
Copy link

Describe the bug

The change introduced in 5b83c01 breaks backwards compability for users that expects the name to be returned.

Not saying this is behavior wanted; but for some reason we had tests that leaned on it (which we will change).

Reproduction

Invoke getFallbackProxy and expect the translation key to be returned.

Logs

No response

Config

No response

Additional information

typesafe-i18n 2.60.1 (commit introduced with said version).

@jbergstroem jbergstroem added the bug Something isn't working label Jan 24, 2022
@ivanhofer
Copy link
Owner

@jbergstroem can you please tell what your tests currently look like and how you plan to change them?
This would help me understand the problem better and I will know if I need to just revert the changes from the fallback proxy or also add some logic to the actual Proxy object.

@jbergstroem
Copy link
Author

@jbergstroem can you please tell what your tests currently look like and how you plan to change them?
This would help me understand the problem better and I will know if I need to just revert the changes from the fallback proxy or also add some logic to the actual Proxy object.

The test essentially relied on getting the key back as a way of validating that an error message was received when a form was posted in a certain way. The author later changed it to lean on another mechanism provided by the validation instead.

@ivanhofer
Copy link
Owner

So you don't actually initialize typesafe-i18n with valid translations inside your tests right?

@ivanhofer
Copy link
Owner

I have reverted the changes made to the fallbackProxy in version 2.60.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants