Skip to content

fix(support): stop unnecessary assignable user fetching - #947

Merged
jakeaturner merged 1 commit into
stagingfrom
fix/assignable-users
Aug 27, 2026
Merged

fix(support): stop unnecessary assignable user fetching#947
jakeaturner merged 1 commit into
stagingfrom
fix/assignable-users

Conversation

@jakeaturner

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI lite review requested due to automatic review settings August 26, 2026 23:59

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown

Severity: LowTicket.tsx:68 prefetches assignable users for every support/harvester ticket, including closed tickets where the picker disables its query. Remove this prefetch or condition it on an editable/open ticket; otherwise the PR retains the unnecessary roster request it aims to eliminate.

@jakeaturner
jakeaturner force-pushed the fix/assignable-users branch from 245f6bb to 44686b9 Compare August 27, 2026 03:54
Copilot AI review requested due to automatic review settings August 27, 2026 03:54

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@jakeaturner
jakeaturner merged commit bb06234 into staging Aug 27, 2026
7 checks passed
@jakeaturner
jakeaturner deleted the fix/assignable-users branch August 27, 2026 04:04
@libretexts-bot

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 2.151.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants