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

Frontend libs 13 migration guide #337

Merged
merged 7 commits into from
Aug 8, 2024

Conversation

goranalkovic-infinum
Copy link
Contributor

Changes:

  • added migration guide for FE libs 13
  • updated dependencies

@goranalkovic-infinum goranalkovic-infinum added dependencies Pull requests that update a dependency file enhancement labels Jul 26, 2024
@goranalkovic-infinum goranalkovic-infinum requested a review from a team July 26, 2024 18:10
@goranalkovic-infinum goranalkovic-infinum self-assigned this Jul 26, 2024

These helpers should be imported from `@eightshift/ui-components/utilities` instead of Frontend libs:
- `camelize` → `camelCase`
- `classnames` → `clsx`
Copy link
Member

Choose a reason for hiding this comment

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

we should make these changes for the libs also?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They'll be made, but that's within FE libs, not docs 😄

dadadavorin
dadadavorin previously approved these changes Jul 29, 2024
Copy link
Contributor

@dadadavorin dadadavorin left a comment

Choose a reason for hiding this comment

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

A huge change indeed, it will be fun times updating this 😅

But awesome work as always 💪

iobrado
iobrado previously approved these changes Jul 29, 2024
## Helpers

### Removed helpers
Some helpers were deprecated and removed, without a replacement. If they were used in the project before, you can take the source of the helper from Frontend libs 12 and included it in your project.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Some helpers were deprecated and removed, without a replacement. If they were used in the project before, you can take the source of the helper from Frontend libs 12 and included it in your project.
Some helpers were deprecated and removed, without a replacement. If they were used in the project before, you can take the source of the helper from Frontend libs 12 and include it in your project.

piqusy
piqusy previously approved these changes Jul 31, 2024
Copy link
Contributor

@piqusy piqusy left a comment

Choose a reason for hiding this comment

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

Nice work 👏🏼

Left a few suggestions

@goranalkovic-infinum goranalkovic-infinum merged commit 0674f52 into main Aug 8, 2024
@goranalkovic-infinum goranalkovic-infinum deleted the feature/fe-libs-13-migration-guide branch August 8, 2024 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants