Skip to content

Conversation

@alrouen
Copy link
Contributor

@alrouen alrouen commented Nov 9, 2024

Add "TotalPages" component into the components export


Important

Add TotalPages to the export list in index.tsx.

  • Exports:
    • Add TotalPages to the export list in index.tsx for external use.

This description was created by Ellipsis for 89c639b. It will automatically update as commits are pushed.

Add "TotalPages" component into the components export
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 89c639b in 19 seconds

More details
  • Looked at 12 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. src/components/index.tsx:77
  • Draft comment:
    Ensure that the TotalPages component is correctly implemented and tested in the Controls/PageNumber module to avoid runtime errors.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The PR correctly adds TotalPages to the export list, which matches the import statement and usage in the Example component.

Workflow ID: wflow_5y4XMZiVKJJPjFmC


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@alrouen
Copy link
Contributor Author

alrouen commented Nov 9, 2024

Hi !
I'm trying to use your viewer into a project. And looking at your source I found this TotalPages component was missing.

@Titou325
Copy link
Member

Titou325 commented Nov 9, 2024

Thank you very much for your contribution! Just merged :)

@Titou325 Titou325 merged commit 2b9fcad into OnedocLabs:main Nov 9, 2024
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