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

refactor(pagination): sync migration guide #22

Merged
merged 1 commit into from
Oct 23, 2023

Conversation

gavin-everett-genesys
Copy link
Collaborator

@gavin-everett-genesys gavin-everett-genesys commented Oct 17, 2023

sync migration guide

Open to changes to the guide :)

COMUI-2263

@github-actions
Copy link

## Overview

- `gux-pagination` is transitioning to `gux-pagination-legacy`
- `gux-pagination-beta` is transitioning to `gux-pagination` which will have a new refreshed design and includes a new property `go-to-page` which allows a user to search for a page click [here](./readme.md)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Have a look at @katie-bobbe-genesys 's gux-modal migration guide in this PR https://github.com/MyPureCloud/genesys-spark/pull/17/files. I think it is a good approach to changes like this.

- </gux-pagination>
+ </gux-pagination-legacy>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we need to add this </gux-pagination-legacy> line here or can it be removed?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I would remove the closing tags from L20 and L21 and keep them here after the ....

- </gux-pagination>
+ </gux-pagination-legacy>
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would remove the closing tags from L20 and L21 and keep them here after the ....


Steps:

- No changes required
Copy link
Collaborator

Choose a reason for hiding this comment

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

The layout options have changed. expanded, full and small -> advanced and simple. I think we recommend changing expanded and full to advanced and small to simple.

sync migration guide

COMUI-2263
@gavin-everett-genesys gavin-everett-genesys merged commit 55d2c2b into beta/v4 Oct 23, 2023
2 checks passed
@gavin-everett-genesys gavin-everett-genesys deleted the feature/COMUI-2263_v2 branch October 23, 2023 14:03
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.

3 participants