[Bug] Analytics passing in entire Nuxt package.json

colbyfayock commented 1 year ago

The analytics ID isn't currently applying correctly, I noticed 2 things:

passing in a non X.X.X version (-rc1) breaks it, which is fine for development builds as its not needed
the analytics object is passing in the entire Nuxt package.json into the configuration as opposed to just the version

Currently:

techVersion: nuxtPkg

Should be:

techVersion: nuxtPkg.version

@nuxtjs/cloudinary 2.0.0-rc.1 nuxt: 3.5.0

colbyfayock commented 1 year ago

i found out there's also a limitation with the techVersion property in that the API only allows a range of 1-10 for minor releases

we can just lock that to major releases and do something along the lines of:

okay - for the techVersion apparently only accepts a minor range of 1-10

so im thinking maybe what we need to do is just limit the techVersion to major

can just use something like this:

`${nuxtPkg.version.split('.')[0]}.0.0`

honestly could do something simiilar with sdkSemver if you think that makes sense

"1.0.0-beta.9".split('-')[0] which would always grab the main bit there

Baroshem commented 1 year ago

nuxt-modules / cloudinary