@convex-dev/expo-push-notifications causing dependency issues with latest Expo SDK ~52.0.19 | Convex Community | Page 1

upbeat quail Dec 18, 2024, 1:56 PM

#

Was confused as to why i was getting this result after running expo-doctor in my project:

And after troubleshooting a bit, and having a talk with AI about it, after sending it a ton of different readouts from npm why etc it ended up saying that it seems like my @convex-dev/expo-push-notifications has mismatch in its depenedencies

I'm fairly sure this doesnt mean i cant export my eas build, as it should be fine. As the component wont ever interfeer with my app native modules.

But i'm sure this would be nice for you guys to know?

The package "expo-modules-autolinking" should not be installed directly in your project. It is a dependency of other Expo packages, which will install it automatically as needed.

Expected package expo-modules-autolinking@~2.0.0
Found invalid:
  expo-modules-autolinking@1.11.3
  (for more info, run: npm why expo-modules-autolinking)
Expected package @expo/config-plugins@~9.0.0
Found invalid:
  @expo/config-plugins@8.0.11
  @expo/config-plugins@8.0.11
  (for more info, run: npm why @expo/config-plugins)
Expected package @expo/prebuild-config@~8.0.0
Found invalid:
  @expo/prebuild-config@7.0.9
  (for more info, run: npm why @expo/prebuild-config)
Expected package @expo/metro-config@~0.19.0
Found invalid:
  @expo/metro-config@0.18.11
  (for more info, run: npm why @expo/metro-config)
Expected package metro@^0.81.0
Found invalid:
  metro@0.80.12
  (for more info, run: npm why metro)
Expected package metro-resolver@^0.81.0
Found invalid:
  metro-resolver@0.80.12
  (for more info, run: npm why metro-resolver)
Expected package metro-config@^0.81.0
Found invalid:
  metro-config@0.80.12
  (for more info, run: npm why metro-config)

#

@convex-dev/expo-push-notifications causing issues with latest Expo SDK ~52.0.19

#

@convex-dev/expo-push-notifications causing dependency issues with latest Expo SDK ~52.0.19

sleek schooner Dec 18, 2024, 2:33 PM

#

I do see that the expo notifications component is at the 0.28 range, guessing expo 52 wants 0.29? But I'm not sure how this output is related

upbeat quail Dec 18, 2024, 7:14 PM

#

I might be wrong then

sleek schooner Dec 18, 2024, 7:15 PM

#

Well, your conclusion makes sense, I'm just saying I don't see the connection with this output if this the only error you're getting

#

This output is telling you not to install expo-modules-autolinking as a direct dependency

#

and that it doesn't like some of your versions, but you can run doctor to fix those

upbeat quail Dec 18, 2024, 8:38 PM

#

Yeah, the output above is from running npx expo-doctor

I've also opened a support request with expo to see if they can help conclude something from this.

I've done following:

cleared npx cache
cleared yarn cache
rm -rf node_modules
delete yarn.lock
yarn expo install --fix

Nothing really seem to change the result from running npx expo-doctor

#

I just noticed something else. Seems like @convex-dev/expo-push-notifications is bringing in the whole tamagui library??

That seems like a mistake

#

Can also confirm that yarn remove @convex-dev/expo-push-notifications fixes the issue above with the dependency issues

sleek schooner Dec 18, 2024, 9:50 PM

#

upbeat quail I just noticed something else. Seems like @convex-dev/expo-push-notifications is...

Ah yeah that's just for the example, should be moved to dev dependencies

sleek schooner Dec 18, 2024, 9:50 PM

#

upbeat quail Can also confirm that yarn remove `@convex-dev/expo-push-notifications` fixes th...

hmmm maybe expo notifications lib should be a peer dep for the component

upbeat quail Dec 18, 2024, 9:50 PM

#

I made an issue on the github repo about the tamagui, so it's not just laying around here

upbeat quail Dec 18, 2024, 9:53 PM

#

sleek schooner hmmm maybe expo notifications lib should be a peer dep for the component

yeah true, and maybe just stating that expo-push-notifications should be installed to use the component in the readme / docs

sleek schooner Dec 18, 2024, 10:07 PM

#

Draft PR: https://github.com/get-convex/expo-push-notifications/pull/7

#

@upbeat quail going to publish a test version can you see if it works for you

upbeat quail Dec 18, 2024, 10:08 PM

#

Ohh thanks Shawn!

#

absolutely

sleek schooner Dec 18, 2024, 10:09 PM

#

npm install @erquhart/expo-push-notifications@0.1.2-github.pr.7

#

It's on my personal for testing purposes, contains the fix in that PR

#

oh wait

#

nope

#

did not build before publishing lol

#

one sec

#

npm install @erquhart/expo-push-notifications@0.1.2-github-pr-7.1

#

That should be good to go, just swap @convex-dev/expo-push-notifications for @erquhart/expo-push-notifications to test after installing wherever you import

upbeat quail Dec 18, 2024, 10:12 PM

#

Alright got it, i was about to clone it and all that haha

#

didnt know i could do yarn install @erquhart/expo-push-notifications@0.1.2-github-pr-7.1

#

u live u learn

sleek schooner Dec 18, 2024, 10:13 PM

#

like specific version?

upbeat quail Dec 18, 2024, 10:14 PM

#

I was just about to clone the repo and checkout ur pr and do it that way

sleek schooner Dec 18, 2024, 10:15 PM

#

aah gotcha

#

that's what I did, then built and published to npm

upbeat quail Dec 18, 2024, 10:15 PM

#

Ahhh i see

sleek schooner Dec 18, 2024, 10:15 PM

#

but I do think you can do it from a github sha as well...actually I probably should have just done that lol

upbeat quail Dec 18, 2024, 10:15 PM

#

I was about to use yarn link and stuff

#

Ive never tried this before, love it

#

😆

sleek schooner Dec 18, 2024, 10:16 PM

#

If we can get this specific build confirmed working for you then that validates the PR I linked above

upbeat quail Dec 18, 2024, 10:19 PM

#

Works like a charm ✨

sleek schooner Dec 18, 2024, 10:19 PM

#

Sweet

#

Can you confirm it actually does what it's supposed to do beyond just not causing errors

#

it has like no dependencies now so I'm half worried I broke something lol

#

(it would have failed ts though if that were the case, just being paranoid)

upbeat quail Dec 18, 2024, 10:21 PM

#

Yes thought the same think, about to do that hahah 👀

#

classic "YUP WORKS NOW"

sleek schooner Dec 18, 2024, 10:21 PM

#

Like, this component doesn't even depend on the expo notifications lib, it's just convex and convex helpers

upbeat quail Dec 18, 2024, 10:22 PM

#

Yeah, so it was all just for the example?

sleek schooner Dec 18, 2024, 10:22 PM

#

So I guess it's just coordinating all of the annoying backend state parts of notifications, but you still do all of the direct expo-notifications bits you normally would

sleek schooner Dec 18, 2024, 10:22 PM

#

upbeat quail Yeah, so it was all just for the example?

yeah seems so

upbeat quail Dec 18, 2024, 10:22 PM

#

Its funny when we installed it, i did not at all look at the dependencies

#

I should do that more often

#

that was like atleast 1 month ago, and the tamagui ui pkg is quite big. prob not the end of the world. But in a big project, every small bit counts i guess

sleek schooner Dec 18, 2024, 10:23 PM

#

no one does and we all should lol

#

i never do

#

nonchalantly shipping tamagui for no reason lol

#

maybe tree shaken out

upbeat quail Dec 18, 2024, 10:24 PM

#

Yeah hopefully HAHA

#

if not we just shipped tamagui in the shadows

sleek schooner Dec 18, 2024, 10:24 PM

#

just in case

#

never know when you'll need the largest ui kit known to man

upbeat quail Dec 18, 2024, 10:25 PM

#

lmao

#

I mean convex dev runs and stuff, testing if the push notifications etc works right now would take me a bit. Since the way we do our builds is that we have different identifiers for development, preview and production builds. And seems like expo push tokens dont register in our system for the environment im in right now, but thats unrelated to the convex component though

#

I can report back in the morning when im at the office, and test this on a preview deployment, but its late now and im heading to bed so out of time

sleek schooner Dec 18, 2024, 10:37 PM

#

ah okay, no problem - no reason it wouldn't work, everything in the component is convex code, nothing external

upbeat quail Dec 18, 2024, 10:37 PM

#

Yeah i totally agree

upbeat quail Dec 18, 2024, 10:53 PM

#

Update: found out i could test in my environment, funny enough what happened was my "sleep" mode has turned on cuz of my sleep schedule on my phone. So the push notification got ignored by my phone. Saw it just now as i got in bed

#

well kudos to apple, that feature certainly works - dangerous for app developers working later than their sleep schedule though 😆

sleek schooner Dec 18, 2024, 10:55 PM

#

lol that's awesome

#

Thanks for the update!

short crypt Dec 19, 2024, 5:57 AM

#

fix released in @convex-dev/expo-push-notifications@0.1.3 thanks!

#@convex-dev/expo-push-notifications causing dependency issues with latest Expo SDK ~52.0.19