Open
Conversation
✅ Deploy Preview for fresco-sandbox ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Deployment failed with the following error: |
🎭 Playwright E2E Test Report❌ Tests failed. View the full report here: |
…ided When createCachedFunction was called without options, the keyParts array defaulted to the tags array only. This missed the VERCEL_DEPLOYMENT_ID prefix that was included when options were provided, causing cache key mismatches between deployments. Also adds comprehensive test coverage for the cache module.
Replace duplicated pnpm/Node.js setup logic across CI workflows with the shared complexdatacollective/github-actions/setup-pnpm composite action. Removes the local .github/actions/pnpm-cache action.
Creates a per-PR deploy preview workflow that: - Creates a Neon database branch for each PR - Sets branch-scoped env vars via Netlify CLI for runtime access - Creates .env for build-time env validation - Deploys to Netlify with PR-specific alias - Comments on PR with preview URL and Neon console link Also adds a cleanup workflow stub for future branch deletion.
- Replace $transaction with Promise.all for read-only queries in
summaryStatistics, activityFeed, and setup page. Prisma driver
adapters implement batch $transaction as interactive transactions
with a 5s default timeout, which Neon cold starts can exceed.
- Remove component reference passed across RSC boundary in
SandboxCredentials (icon={KeyRound} passed from server component
to client Alert component).
initial-setup.spec.ts and onboarding.spec.ts both tested the same setup wizard flow. Since they ran sequentially in the setup project, the first spec configured the app, causing the second to redirect to /signin instead of /setup.
|
Contributor
7 tasks
Contributor
|
use participant label (if available) and identifier
should not be readonly
- Split workflow into check-pr and deploy-preview jobs; skip deploy when no PR exists (fixes pr--branchname duplicates) - Add concurrency group to cancel in-progress runs on same branch - Update existing PR comment instead of creating new ones each push - Add 15 minute timeout to deploy job - Use pnpm dlx for netlify-cli instead of global install - Add graceful error handling when cleanup branch doesn't exist - Remove unnecessary permissions from both workflows Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
include deploy log, commit
creates consistency with including ego in naming step
use target stage prompt count instead of current stage when calculating progress during navigation animation
use inline svg rather than the icon component bc existing icon hardcodes body to be neon-coral. this approach allows for variants for the two bgs
handled outside of this pr
…oads fixes presigned upload on firefox
…stration, clean up
Create dedicated PreviewProtocol table to avoid hash conflicts with regular protocols, return presigned urls with assetid to fix asset duplication bug
Pedigree bugfixes
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Branch for use by the family pedigree project.