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: Add pagination total results #383

Merged
merged 3 commits into from
May 2, 2024

Conversation

b-hok
Copy link
Contributor

@b-hok b-hok commented Apr 23, 2024

What this PR does / why we need it:
Formats the total number of items of the pagination result according to the user's preferred language. Applied to locales/en/pagination

Which issue(s) this PR closes:

Closes #330

Special notes for your reviewer:

Suggestions on how to test this:

Storybook (There are additional areas that can be viewed too)
Sample URL 1: http://localhost:6006/?path=/story/sections-dataset-page-datasetfiles--no-filters

Cypress
Relating tests found in PaginationResultsInfo.spec.tsx

Does this PR introduce a user interface change? If mockups are available, please link/include them here:
See Issue #330

Is there a release notes update needed for this change?:

Additional documentation:

@ekraffmiller ekraffmiller added pm.GREI-d-2.7.1 NIH, yr2, aim7, task1: R&D UI modules for creating datasets and supporting publishing workflows pm.GREI-d-2.7.2 NIH, yr2, aim7, task2: Implement UI modules for creating datasets and publishing workflows UI Tasks related to the user interface (UI) or frontend development labels Apr 24, 2024
@g-saracca g-saracca self-assigned this Apr 25, 2024
Copy link
Contributor

@g-saracca g-saracca left a comment

Choose a reason for hiding this comment

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

Hi @b-hok, many thanks for your contribution !
It looks great overall, I just have a few comments to share with you.

@g-saracca g-saracca assigned b-hok and unassigned g-saracca Apr 25, 2024
@g-saracca g-saracca assigned g-saracca and unassigned b-hok Apr 26, 2024
@g-saracca g-saracca assigned b-hok and unassigned g-saracca Apr 29, 2024
Copy link
Contributor

@g-saracca g-saracca left a comment

Choose a reason for hiding this comment

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

LGTM now! Approved!
We will move to QA this issue and merge it later if everything is working as expected!
Many thanks for your contribution to Dataverse Frontend! 🙌

@g-saracca g-saracca added the Size: 3 A percentage of a sprint. 2.1 hours. label Apr 29, 2024
@GPortas GPortas self-assigned this May 1, 2024
@GPortas GPortas merged commit 0feb0ae into IQSS:develop May 2, 2024
jayanthkomarraju pushed a commit to jayanthkomarraju/dataverse-frontend that referenced this pull request May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pm.GREI-d-2.7.1 NIH, yr2, aim7, task1: R&D UI modules for creating datasets and supporting publishing workflows pm.GREI-d-2.7.2 NIH, yr2, aim7, task2: Implement UI modules for creating datasets and publishing workflows Size: 3 A percentage of a sprint. 2.1 hours. UI Tasks related to the user interface (UI) or frontend development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Format total in table pagination results according to browser language.
4 participants