Closed CarolinaCobo closed 1 month ago
@CarolinaCobo is attempting to deploy a commit to the Codú Team on Vercel.
A member of the Team first needs to authorize it.
[!IMPORTANT]
Review skipped
Review was skipped due to path filters
:no_entry: Files ignored due to path filters (1)
* `.github/workflows/pull-request.yml` is excluded by `!**/*.yml`CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including
**/dist/**
will override the default block on thedist
directory, by removing the pattern from both the lists.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
The changes in the pull request involve significant modifications to the server/db/schema.ts
file, focusing on the addition of new database tables and their relationships. Four new tables—Community
, Membership
, RSVP
, and Event
—are introduced, each with specific fields and relationships to other tables. The schema now supports enhanced functionality for managing communities, events, memberships, and RSVPs, along with the inclusion of a new import statement for the crypto
module.
File | Change Summary |
---|---|
server/db/schema.ts | Added new tables: Community , Membership , RSVP , Event . Established relationships: communityRelations , membershipRelations , r_s_v_pRelations , eventRelations . New import for crypto module. |
.nvmrc | Version updated from v18.17.1 to v20.17.0 . |
🐇 In the garden where new tables bloom,
Communities gather, dispelling the gloom.
With events and memberships, joy takes flight,
RSVP, oh what a wonderful sight!
In our schema, connections grow wide,
Together we flourish, side by side! 🌼
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
I cant see crypto in the package.json ?
Can you import crypto directly like this from Node V18 ?
you don't need it on the package, if you add it on the server/db/schema.ts at the top it will work. I was talking to Niall about it and it works no problem in node 20 so we'll be updating that too
you don't need it on the package, if you add it on the server/db/schema.ts at the top it will work. I was talking to Niall about it and it works no problem in node 20 so we'll be updating that too
Really interesting! Is it bundled with Node v18?
Edit: Bot answered my question. Its bundled in Node versions >=14.17.0
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
codu | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Oct 2, 2024 8:47pm |
✨ Codu Pull Request 💻
Fixes #(issue)
Pull Request details
npm run db:seed
and getting this errorError: ReferenceError: crypto is not defined
Any Breaking changes
Associated Screenshots