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

add stories to roles components #10503

Merged
merged 4 commits into from
Feb 26, 2025
Merged

Conversation

Weiko
Copy link
Member

@Weiko Weiko commented Feb 26, 2025

Context

Adding stories for roles components. Also moving modules components to the proper "modules" folder, "pages" folder being only for entry points.

Test

Run storybook

Screenshot 2025-02-26 at 13 40 40 Screenshot 2025-02-26 at 13 40 32 Screenshot 2025-02-26 at 13 40 11 Screenshot 2025-02-26 at 13 38 16 Screenshot 2025-02-26 at 13 38 12

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

Added Storybook stories for roles components and reorganized the component structure by moving role-related components from 'pages' to 'modules' directory.

  • Added stories for Roles, RolesDefaultRole, SettingsRoles, and SettingsRoleEdit components with appropriate decorators and mock data
  • Created new mock data in /packages/twenty-front/src/testing/mock-data/roles.ts with Admin and Guest role examples
  • Added GraphQL mock for GET_ROLES query in graphqlMocks.ts to support Storybook stories
  • Updated import paths across multiple files from ~/pages/settings/roles/... to @/settings/roles/...
  • Made colorScheme property required in WorkspaceMember.ts type definition

Greptile AI

29 file(s) reviewed, 4 comment(s)
Edit PR Review Bot Settings | Greptile

@Weiko Weiko merged commit 431da37 into main Feb 26, 2025
45 of 46 checks passed
@Weiko Weiko deleted the c--add-stories-to-roles-components branch February 26, 2025 17:16
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