-
Notifications
You must be signed in to change notification settings - Fork 8
page CT added #682
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
page CT added #682
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds support for a new "Pages" content type and adjusts configuration to support a migration from a Contentful CMS to Wordpress.
- Updated configuration to change the CMS type to Wordpress and set an absolute path for legacy content.
- Added a new "Pages" schema in the content types library with associated fields.
- Extended migration services and constants to support extraction and processing of Pages.
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| upload-api/src/config/index.ts | Updated CMS type to Wordpress and set a hard-coded localPath for legacy content. |
| upload-api/migration-wordpress/libs/content_types.js | Added new Pages content type schema with various fields and options. |
| api/src/services/migration.service.ts | Integrated extractPages calls in both testing and production migration workflows. |
| api/src/constants/index.ts | Added constants definitions for Pages directories and filenames. |
| .talismanrc | Updated configuration ignore patterns and checksum entries. |
Co-authored-by: Copilot <[email protected]>
|
|
||
| // iterate over data of each file | ||
| for (const data of chunkData) { | ||
| const filteredChunk = chunkData.filter((item:any) => item["wp:post_type"] === "post" && ["publish", "inherit", "draft"].includes(item["wp:status"])); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add ?. for filter and includes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't commit this file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolved
No description provided.