search

bwgjoseph / mongoose-vs-ottoman

feature comparison between mongoose and ottoman
0 stars 1 forks source link

New Beta testing - GA Readiness #109

Closed AV25242 closed 2 years ago

AV25242 commented 2 years ago

Hello @bwgjoseph , we released new beta today.

The request is if you and team can do the regression just like you do always and help us release a GA by end of them month. We are targeting GA end of the month also depends if nothing major was found (which we hope not).

Can you assist please ?

bwgjoseph commented 2 years ago

Sure, but would have to be sometime later this week/weekend

AV25242 commented 2 years ago

Yeah thanks, appreciate your quick response. Sooner will help us mitigate critical issues if any and do a GA possibly

bwgjoseph commented 2 years ago

So I'm trying to upgrade to beta.9 and test out the features, but I've encountered some issue immediately after upgrading.

So this is at beta.8, running this specific test case and it works just fine

image

Once I upgrade to beta.9, it throws me this when I run the same test

image

And that is referring to the query method in this method which I have no idea why it throws error.

So to try to continue, I removed that method, and I face with yet another issue

image

Which I have no clue what is causing it. I did tried setting the scope but it didn't work too.

new Ottoman({ scopeName: '_default', collectionName: '_default' });

I can't continue to test it for now, until I figured out something myself later on, or if you could provide me with some solution/workaround?

Also, as there are quite a bit of changes since beta.8, I'm not sure what has changed, and there's no changelog, and it's a little bit difficult to trace.

AV25242 commented 2 years ago

Thanks Joseph, I think this is a more common question asked and the answer is that ottoman.connect is now async the docs have been updated to reflect this change https://ottomanjs.com/#installation

I 100% agree that there is no change-log but rest assured we are working on getting it and it will be there with release post GA

bwgjoseph commented 2 years ago

Thanks, adding the await seem to solved it. So the query issue is also because there's no connection, hence, the error.

Should have crossed my mind, I remember seeing that changes.

I should be able to proceed to test the rest of it now. Thanks!