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

Resolve issue exporting & importing booleans #14

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

Conversation

aryehraber
Copy link
Contributor

This PR tackles #11 by casting booleans as strings during export, and then back to booleans during import.

Let me know if you have any questions!

@mattias-persson
Copy link
Owner

@aryehraber Exporting works great! However, I'm having trouble importing falsy values. For example, I have a toggle on a page which is currently enabled, I export and change the value from true to false and then import it again. The toggle doesn't seem to change.

@aryehraber
Copy link
Contributor Author

@mattias-persson Thanks for testing and the feedback -- can definitely take another look!

Before I do, just want to double check: did you check the translated version for the toggle change or the original? I kept making this mistake while testing 🙈

@mattias-persson
Copy link
Owner

@aryehraber Pretty sure I tried both! When only changing the target value to false nothing happened 🙂

@aryehraber
Copy link
Contributor Author

@mattias-persson Cool, thanks for confirming. Will run some more tests later this week and report back!

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