-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
Author and diff tracking #395
Conversation
🦋 Changeset detectedLatest commit: d62500c The changes in this PR will be included in the next version bump. This PR includes changesets to release 12 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 |
Thanks for opening this pull request! 🎉 The following links will become available for preview shortly (if applicable): TEMPORARILY DISABLED: The StudioCMS Demo (playgrounds/node) link is currently disabled due to ongoing development on the next version. We apologize for the inconvenience and appreciate your understanding. 🙏
Note: Changes may take a few minutes to propagate. Please be patient! You can also see the status of each deployment via the withstudiocms-deployments bot 🤖 comments. Please review the changes and provide any feedback. Thanks! 🚀 |
studiocms
@studiocms/assets
@studiocms/betaresources
@studiocms/auth
@studiocms/blog
@studiocms/core
@studiocms/dashboard
@studiocms/devapps
@studiocms/frontend
@studiocms/imagehandler
@studiocms/renderers
@studiocms/robotstxt
commit: |
…ty and performance
@@ -141,5 +141,7 @@ export const StudioCMSSiteConfig = defineTable({ | |||
siteIcon: column.text({ optional: true }), | |||
loginPageBackground: column.text({ default: 'studiocms-curves' }), | |||
loginPageCustomImage: column.text({ optional: true }), | |||
enableDiffs: column.boolean({ default: false }), | |||
diffPerPage: column.number({ default: 10 }), |
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.
This should probably be diffsPerPage
?
diffPerPage: column.number({ default: 10 }), | |
diffsPerPage: column.number({ default: 10 }), |
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.
Valid point. Only downside... its a bit hard to rename a table column after you've already pushed the schema to the database... 😅 and we would probably need the first time setup stuff ready first... so i can purge the db and reset while updating the schema
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.
Awesome work!
This pull request includes significant changes to the
StudioCMS
project, focusing on adding diff tracking capabilities, updating dependencies, and making layout adjustments. The most important changes include adding new diff tracking features, updating package dependencies, and modifying layout files.Diff Tracking Features:
enableDiffs
anddiffPerPage
to theStudioCMSSiteConfig
table (packages/studiocms_core/src/db/tables.ts
).packages/studiocms_core/src/lib/routeMap.ts
).StudioCMSSDK
class, including methods to insert, clear, get, and revert diffs (packages/studiocms_core/src/sdk-utils/StudioCMSSDK.ts
).Dependency Updates:
diff
,diff2html
, and their respective type definitions topackage.json
(packages/studiocms_core/package.json
). [1] [2]Layout Adjustments:
FallbackCanvas.astro
by closing adiv
tag and removing unnecessary whitespace (packages/studiocms_auth/assets/layouts/FallbackCanvas.astro
). [1] [2] [3]diff.css
inBaseHead.astro
for styling diffs (packages/studiocms_dashboard/assets/components/BaseHead.astro
). [1] [2]Changes and Notes:
Adds Author and Contributor Tracking to pages
Implements new Diff system
/content-management/diff
New content previews:
closes #187
closes #293