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

Base64 for binary fields #319

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Jaza
Copy link

@Jaza Jaza commented Jul 29, 2024

For fields that contain non-string binary data (e.g. random tokens), base64 encode the value for display in list views and in forms, and base64 decode the value on save.

It's unlikely that you'll want such fields to actually have their value be visible and editable in kaffy. But all fields are visible and editable by default in kaffy, and we don't want a broken out-of-the-box experience for any field types. And, in case you do want to work with such fields, they should be visible in a legible / useful format, and they should be editable without the value getting corrupted on save.

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.

1 participant