feat(cms): implement file area block and seed a dedicated page#316
feat(cms): implement file area block and seed a dedicated page#316
Conversation
|
|
There was a problem hiding this comment.
Pull Request Overview
This PR implements functionality to support a dedicated file area block and seeds a dedicated page for tenants with the file-area tag. It updates several seed functions and collection configurations, adds new migrations for tag and file-area support, and adjusts the reset-db process to integrate media deletion.
Reviewed Changes
Copilot reviewed 99 out of 99 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| libs/app-cms/feature/seed/src/lib/local-api/ensure-post.ts | Updated return type to remove string values, ensuring consistent numeric IDs. |
| libs/app-cms/feature/seed/src/lib/local-api/ensure-page.ts | Adjusted return type to match numeric ID pattern. |
| libs/app-cms/feature/seed/src/lib/local-api/ensure-navigation.ts | Modified navigation seeding logic to merge missing items and return an object with navigation ID and new items. |
| libs/app-cms/feature/seed/src/lib/local-api/ensure-media.ts | Added file existence check and improved media lookup. |
| libs/app-cms/feature/seed/src/lib/custom-seed.ts | Added custom seed logic for file-area pages and navigation. |
| apps/cms/src/utils/reset-db.ts | Updated reset workflow to handle media deletion and use Drizzle adapter for DB drop. |
| apps/cms/src/payload.config.ts | Expanded block registrations to include file-area and image blocks. |
| apps/cms/src/migrations/* | Introduced new migration for COD-292 with new tables, constraints, and indices. |
| apps/cms/src/collections/* | Modified collections (tags, posts, pages, media, categories, reusable-content) to support new file-area functionality and adjust admin columns. |
| apps/cms/.env.local | Updated Ethereal email credentials for local development. |
Comments suppressed due to low confidence (5)
libs/app-cms/feature/seed/src/lib/local-api/ensure-post.ts:23
- The removal of the string type from the return type may affect clients expecting string IDs; please verify that all consuming code has been updated accordingly.
): Promise<Post | number> {
libs/app-cms/feature/seed/src/lib/local-api/ensure-page.ts:24
- Ensure that changing the return type to only allow numeric IDs does not break any existing integrations or tests that relied on a string type being returned.
): Promise<Page | number> {
libs/app-cms/feature/seed/src/lib/local-api/ensure-category.ts:19
- Confirm that the removal of string from the return type is intentional and that all corresponding usages in the codebase are adjusted to expect a numeric ID.
): Promise<Category | number> {
apps/cms/src/collections/posts/posts.collection.ts:45
- [nitpick] The removal of the 'tenant' column from the defaultColumns array may impact admin displays; please confirm that this adjustment aligns with the intended admin UI design.
defaultColumns: ['title', 'updatedAt'],
apps/cms/src/collections/categories/categories.collection.ts:17
- [nitpick] Removing the 'tenant' column from the defaultColumns array should be verified to ensure the admin interface still provides all necessary context for category records.
defaultColumns: ['name', 'slug'],
|
View your CI Pipeline Execution ↗ for commit efe4d5f
☁️ Nx Cloud last updated this comment at |
|
View your CI Pipeline Execution ↗ for commit fa75240.
☁️ Nx Cloud last updated this comment at |
1 similar comment
|
View your CI Pipeline Execution ↗ for commit fa75240.
☁️ Nx Cloud last updated this comment at |
|
✨ Your pull request projects are ready for preview
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #316 +/- ##
=======================================
Coverage 75.26% 75.26%
=======================================
Files 61 61
Lines 1047 1047
Branches 224 224
=======================================
Hits 788 788
Misses 259 259 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
✨ Your pull request projects are ready for preview
|
|
✨ Your pull request projects are ready for preview
|
|
✨ Your pull request projects are ready for preview
|
hakalb
force-pushed
the
cod-292-filearea
branch
from
ede332f to
89297cd
Compare
|
✨ Your pull request projects are ready for preview
|
hakalb
force-pushed
the
cod-292-filearea
branch
from
89297cd to
a030553
Compare
|
✨ Your pull request projects are ready for preview
|
hakalb
force-pushed
the
cod-292-filearea
branch
from
a030553 to
11ce9c0
Compare
|
✨ Your pull request projects are ready for preview
|
hakalb
force-pushed
the
cod-292-filearea
branch
2 times, most recently
from
17a9108 to
611a916
Compare
|
✨ Your pull request project is ready for preview
|
hakalb
force-pushed
the
cod-292-filearea
branch
from
611a916 to
c18bc5d
Compare
|
✨ Your pull request project is ready for preview
|
|
✨ Your pull request project is ready for preview
|
|
✨ Your pull request project is ready for preview
|
|
✨ Your pull request project is ready for preview
|
|
✨ Your pull request project is ready for preview
|
hakalb
force-pushed
the
cod-292-filearea
branch
from
2946bd3 to
482e1db
Compare
|
✨ Your pull request project is ready for preview
|
|
✨ Your pull request projects are ready for preview
|
hakalb
force-pushed
the
cod-292-filearea
branch
from
b586c2c to
efe4d5f
Compare
|
✨ Your pull request projects are ready for preview
|
closed COD-292