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

Migrate to twenty-ui - feedback/progress-bar #8002

Merged
merged 2 commits into from
Oct 24, 2024
Merged

Migrate to twenty-ui - feedback/progress-bar #8002

merged 2 commits into from
Oct 24, 2024

Conversation

gitstart-app[bot]
Copy link
Contributor

@gitstart-app gitstart-app bot commented Oct 23, 2024

This PR was created by GitStart to address the requirements from this ticket: TWNTY-7527.


Description

Migrate feedback/progress-bar to twenty ui:

  • CircularProgressBar
  • ProgressBar

Demo

ProgressBar on Storybook

CircularProgressBar on Storybook

Fixes twentyhq/private-issues#91

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

This pull request migrates the CircularProgressBar and ProgressBar components from the local UI folder to the twenty-ui library, ensuring consistent functionality across the codebase.

  • Updated import statements in multiple files to use 'twenty-ui' for CircularProgressBar and ProgressBar
  • Moved CircularProgressBar.tsx and ProgressBar.tsx to packages/twenty-ui/src/feedback/progress-bar/components/
  • Added new index.ts file in packages/twenty-ui/src/feedback/ to export progress bar components
  • Updated Storybook stories for CircularProgressBar and ProgressBar in twenty-ui package
  • Removed duplicate Button component export in packages/twenty-front/tsup.ui.index.tsx

12 file(s) reviewed, 5 comment(s)
Edit PR Review Bot Settings | Greptile

@@ -23,6 +21,7 @@ export * from './src/modules/ui/input/components/IconPicker';
export * from './src/modules/ui/input/components/ImageInput';
export * from './src/modules/ui/input/components/Radio';
export * from './src/modules/ui/input/components/RadioGroup';
export * from './src/modules/ui/input/button/components/Button';
Copy link
Contributor

Choose a reason for hiding this comment

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

syntax: Remove this duplicate export of Button component

@charlesBochet charlesBochet merged commit 445ab83 into main Oct 24, 2024
18 checks passed
@charlesBochet charlesBochet deleted the TWNTY-7527 branch October 24, 2024 12:05
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