fix(integration): setup #5511
fix(integration): setup #5511
Conversation
🦋 Changeset detectedLatest commit: e6bf130 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 3 Ignored Deployments
|
|
@riqwan @pKorsholm @olivermrbl could you have a look please. I a currently moving some fixed over there instead of my other pr about the medusa app loader |
| @@ -35,6 +35,10 @@ class RedisCacheService implements ICacheService { | |||
| data: Record<string, unknown>, | |||
| ttl: number = this.TTL | |||
| ): Promise<void> { | |||
| if (ttl === 0) { | |||
There was a problem hiding this comment.
note to self: breaking change
There was a problem hiding this comment.
Added one question in there, which is pretty important if its not being done at the moment. We need to be able to use verbose: true when we need to see whats happening deep in the modules.
This also now breaks the plugins/__tests__/product/admin/* specs, which wasn't happening before.
| @@ -168,6 +174,7 @@ module.exports = { | |||
| const options = { | |||
| database: { | |||
| clientUrl: DB_URL, | |||
| connection: pgConnection, | |||
No description provided.