Insights: fixed for local development and q-manifest submissions #7642
Conversation
🦋 Changeset detectedLatest commit: c99dc59 The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 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 |
|
| @@ -99,9 +99,10 @@ export const symbolDetailTable = sqliteTable( | |||
| hi: integer('hi').notNull(), | |||
| }, | |||
| (symbolDetailTable) => ({ | |||
| apiKeyManifestHashIndex: uniqueIndex('idx_symbolDetail_apiKey_manifestHash').on( | |||
| apiKeyManifestHashIndex: uniqueIndex('idx_symbolDetail_apiKey_manifestHash_hash').on( | |||
There was a problem hiding this comment.
@mhevery please look at this and see if it's OK.
because I remember us having issues with the indexing.
This specifically fixes an sql error when submitting a q-manifest via the vite plugin
shairez
force-pushed
the
pr-fix-insights-app
branch
from
72d996d to
c99dc59
Compare
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
wmertens
changed the title
What is it?
Description
Fixed the insights app for local development and also q-manifest sending.
Improved the design a bit by using qwik-ui tooltip
(for now I kept it 0.5.0 because the docs use the same version, but we'll upgrade later and make the wrapper with less
any:) )Checklist
pnpm change