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

Merged
merged 2 commits into from
Jul 29, 2024

Conversation

arnauddorgans
Copy link
Contributor

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.

@BobaFetters BobaFetters requested a review from a team as a code owner November 15, 2023 18:13
@calvincestari calvincestari removed the request for review from AnthonyMDev November 15, 2023 19:22
@AndersonMartins1
Copy link

Do you need help with something?

Copy link

netlify bot commented Jul 29, 2024

👷 Deploy request for apollo-ios-docc pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit ae78d97

Copy link

netlify bot commented Jul 29, 2024

👷 Deploy request for eclectic-pie-88a2ba pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit ae78d97

@calvincestari calvincestari merged commit 17d905d into apollographql:main Jul 29, 2024
23 checks passed
BobaFetters pushed a commit that referenced this pull request Jul 29, 2024
BobaFetters pushed a commit to apollographql/apollo-ios that referenced this pull request Jul 29, 2024
BobaFetters pushed a commit that referenced this pull request Jul 29, 2024
9d0bb802 fix Array JSONEncodable (#128)

git-subtree-dir: apollo-ios
git-subtree-split: 9d0bb802d9eb8aaa9ede384461824779807dff0b
BobaFetters pushed a commit that referenced this pull request Jul 29, 2024
git-subtree-dir: apollo-ios
git-subtree-mainline: 745fdc9
git-subtree-split: 9d0bb802d9eb8aaa9ede384461824779807dff0b
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.

4 participants