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

User Portal agent listing improvements #2108

Merged
merged 2 commits into from
Jan 2, 2025
Merged

Conversation

joelhulen
Copy link
Contributor

@joelhulen joelhulen commented Jan 1, 2025

User Portal agent listing improvements

The issue or feature being addressed

Fixes issues related to the order in which the User Portal retrieves agents and conversations (sessions). Issues include:

  • Improperly deriving the agent and setting it to the selected agent when refreshing the browser while looking at a specific conversation
  • Allowing the user to enter a prompt before the list of agents is available when there is a large number of conversations
  • Perceived delay in populating the agents list on page load

Details on the issue fix or feature implementation

  • Moved agent loading method call from the NavBar component to the init method of the app store
  • Ensured the agent loading method call is moved to the top of the init method before session-loading logic occurs
  • Updated the NavBar component to construct the agent options when the app store's agents collection has changed, rather than when the component is created

Confirm the following

  • I started this PR by branching from the head of the default branch
  • I have targeted the PR to merge into the default branch
  • This PR needs to be cherry-picked into at least one release branch
  • I have included unit tests for the issue/feature
  • I have included inline docs for my changes, where applicable
  • I have successfully run a local build
  • I have provided the required update scripts, where applicable
  • I have updated relevant docs, where applicable

Note

Instead of adding X's inside the checkboxes you wish to check above, first submit the PR, then check the boxes in the rendered description.

@ciprianjichici ciprianjichici merged commit cc4f95b into main Jan 2, 2025
9 checks passed
@ciprianjichici ciprianjichici deleted the jdh-user-portal-agents branch January 2, 2025 14:47
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