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

fix Array JSONEncodable #128

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

Conversation

arnauddorgans
Copy link

Avoid using fatal error on JSONEncodable instead of conforming protocol with where clause

@AnthonyMDev
Copy link
Contributor

Hmmm... I wanted to do this ages ago, and there was some reason why I couldn't. I think it was causing some build errors.

We're going to need to test this manually on iOS versions pre 14.4. I'm wondering if the issues I saw before had to do with the conversion of the nested AnyHashable types inside of the array (which doesn't work properly on iOS < 14.4).

Thanks for the PR. Give me some time to look into this a bit further before we pull it in.

@AndersonMartins1
Copy link

Do you need help with something?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants