Skip to content

[instrument_manager] Page count fix#10655

Merged
driusan merged 9 commits into
aces:29.0-releasefrom
fias06:2026_06_18_fix_instrument_manager_page_count
Jun 30, 2026
Merged

[instrument_manager] Page count fix#10655
driusan merged 9 commits into
aces:29.0-releasefrom
fias06:2026_06_18_fix_instrument_manager_page_count

Conversation

@fias06

@fias06 fias06 commented Jun 18, 2026

Copy link
Copy Markdown

Brief summary of changes

Fixed the page count on instrument manager

  • Have you updated related documentation?
Screenshot 2026-06-18 at 2 08 56 PM

Testing instructions (if applicable)

  1. Go to instrument manager page
  2. See that the number appears

Link(s) to related issue(s)

@fias06 fias06 self-assigned this Jun 18, 2026
@github-actions github-actions Bot added the Language: Javascript PR or issue that update Javascript code label Jun 18, 2026
@fias06 fias06 force-pushed the 2026_06_18_fix_instrument_manager_page_count branch from db2f811 to eee4518 Compare June 18, 2026 18:26
@fias06 fias06 force-pushed the 2026_06_18_fix_instrument_manager_page_count branch from 1fbc053 to 48454e0 Compare June 18, 2026 19:52
@fias06 fias06 added Module: instrument_manager PR or issue related to instrument_manager module 29.0.0 - Bugs Issues found during the release testing for 29.0.0 Critical to release PR or issue is key for the release to which it has been assigned labels Jun 19, 2026

@lapadulamichael lapadulamichael left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would suggest simply adding import i18n from 'I18nSetup'; to instrumentManagerIndex.js instead of changing DataTable.js

@lapadulamichael lapadulamichael added the State: Needs work PR awaiting additional work by the author to proceed label Jun 26, 2026
@fias06 fias06 removed the State: Needs work PR awaiting additional work by the author to proceed label Jun 29, 2026
Comment thread jsx/DataTable.js Outdated
@fias06 fias06 requested a review from lapadulamichael June 29, 2026 20:14
@fias06 fias06 assigned lapadulamichael and unassigned fias06 Jun 29, 2026

@lapadulamichael lapadulamichael left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@lapadulamichael lapadulamichael added the Passed manual tests PR has been successfully tested by at least one peer label Jun 29, 2026
@driusan driusan merged commit 383a94d into aces:29.0-release Jun 30, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

29.0.0 - Bugs Issues found during the release testing for 29.0.0 Critical to release PR or issue is key for the release to which it has been assigned Language: Javascript PR or issue that update Javascript code Module: instrument_manager PR or issue related to instrument_manager module Passed manual tests PR has been successfully tested by at least one peer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants