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

deps(coral): Update design-system #2535

Merged
merged 5 commits into from
Aug 22, 2024
Merged

deps(coral): Update design-system #2535

merged 5 commits into from
Aug 22, 2024

Conversation

programmiri
Copy link
Contributor

Linked issue

Updates design system and related needed changes.

What kind of change does this PR introduce?

  • Bug fix
  • New feature
  • Refactor
  • Docs update
  • CI update
  • [ x] Deps update

What is the current behavior?

Describe the state of the application before this PR. Illustrations appreciated (videos, gifs, screenshots).

What is the new behavior?

Describe the state of the application after this PR. Illustrations appreciated (videos, gifs, screenshots).

Other information

Additional changes, explanations of the approach taken, unresolved issues, necessary follow ups, etc.

Requirements (all must be checked before review)

  • The pull request title follows our guidelines
  • Tests for the changes have been added (if relevant)
  • The latest changes from the main branch have been pulled
  • pnpm lint has been run successfully

@programmiri programmiri self-assigned this Jul 26, 2024
aindriu-aiven
aindriu-aiven previously approved these changes Aug 2, 2024
Copy link
Contributor

@aindriu-aiven aindriu-aiven left a comment

Choose a reason for hiding this comment

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

LGTM

Built and tested didnt see any issues.

@programmiri programmiri force-pushed the update-ds branch 4 times, most recently from 3cf945d to babb8ae Compare August 7, 2024 14:38
@programmiri programmiri force-pushed the update-ds branch 3 times, most recently from 8d07426 to 4cad1ac Compare August 22, 2024 11:03
programmiri and others added 5 commits August 22, 2024 13:25
Signed-off-by: Mirjam Aulbach <[email protected]>
Signed-off-by: Mirjam Aulbach <[email protected]>
Signed-off-by: Mirjam Aulbach <[email protected]>
Signed-off-by: Mirjam Aulbach <[email protected]>
aindriu-aiven
aindriu-aiven previously approved these changes Aug 22, 2024
Copy link
Contributor

@aindriu-aiven aindriu-aiven left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@aindriu-aiven aindriu-aiven left a comment

Choose a reason for hiding this comment

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

LGTM

@aindriu-aiven aindriu-aiven merged commit 29e5adc into main Aug 22, 2024
26 checks passed
@aindriu-aiven aindriu-aiven deleted the update-ds branch August 22, 2024 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants