Skip to content
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

Restore "Fix "Create profile" page not supporting dark mode (#8110)" & Fix string defaultValues during sync-metadata #9220

Merged
merged 3 commits into from
Dec 24, 2024

Conversation

Weiko
Copy link
Member

@Weiko Weiko commented Dec 23, 2024

Restoring #9185
Also fixing sync-metadata with test values in jsonb

Test

sync-metadata on existing workspaces should replace colorSchema in both metadata and workspaceMember tables

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Changes default color scheme from 'Light' to 'System' in workspace member creation to support dark mode functionality on the 'Create profile' page.

  • Modified default colorScheme value in /packages/twenty-server/src/modules/workspace-member/standard-objects/workspace-member.workspace-entity.ts
  • Todo: Fix sync-metadata with JSON defaultValue before merging
  • Impacts new user onboarding experience by respecting system color scheme preferences

2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@Weiko Weiko requested a review from guillim December 24, 2024 13:37
@Weiko Weiko changed the title Restore "Fix "Create profile" page not supporting dark mode (#8110)" Restore "Fix "Create profile" page not supporting dark mode (#8110)" & Fix string defaultValues during sync-metadata Dec 24, 2024
Copy link
Contributor

@guillim guillim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Weiko Weiko merged commit 2dcfaea into main Dec 24, 2024
22 checks passed
@Weiko Weiko deleted the revert-9201-revert-9185-fix/8110-new-user-dark-mode branch December 24, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants