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

feat: migration guide #3045

Merged
merged 4 commits into from
Feb 24, 2023
Merged

Conversation

aluarius
Copy link
Contributor

Description

  • migrations guide for docs.

Types of changes

  • New feature (non-breaking change which adds functionality)

@aluarius aluarius requested review from asvae and m0ksem February 22, 2023 12:31
@aluarius aluarius self-assigned this Feb 22, 2023
"image": "Component was reworked. Suggest to check all places of usage.",
"extensions": "Extensions",
"aggrid_0": "Ag Grid",
"aggrid_1": "Removed SCSS mixins for Vuestic UI theme customization.",
Copy link
Collaborator

Choose a reason for hiding this comment

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

They must be available with @vuestic/ag-grid/scss import. @rustem-nasyrov?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This phrase was approved by him.

"extensions": "Extensions",
"aggrid_0": "Ag Grid",
"aggrid_1": "Removed SCSS mixins for Vuestic UI theme customization.",
"tailwind": "Added {'@vuestic/tailwind'} package for config syncing between Vuestic UI and Tailwind CSS."
Copy link
Collaborator

Choose a reason for hiding this comment

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

"Deprecated grid. Use Tailwind instead. "

It is migration guide, not release log.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll change, but imo migration guide should contain something like do it, or you'll face issues in this version. Unlikely grid system in 1.6.0.

@aluarius aluarius requested a review from m0ksem February 23, 2023 08:28
@m0ksem m0ksem merged commit 644c4a0 into epicmaxco:develop Feb 24, 2023
@m0ksem m0ksem added the docs packages/docs label Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs packages/docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants