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

Enhancement: Ability to order assistants by order of specified supported id's #4542

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dannypbroadbent
Copy link

@dannypbroadbent dannypbroadbent commented Oct 24, 2024

Summary

Enable the option orderBySupportIdsFirst to true to order assistants by specified supported id's, useful if you want to add your own, and would like them listed first (first one specified will become the main assistant by default).

Screenshot 2024-10-24 at 16 54 13

Change Type

  • New feature (non-breaking change which adds functionality)

Testing

Enable / disable the config orderBySupportIdsFirst to see the change happen, you can also swap supported id's around to see the ordering change.

NOTE: I've noticed you need to clear the app data with a fresh login to see the new ordering appear.

Test Configuration:

Checklist

Please delete any irrelevant options.

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have commented in any complex areas of my code
  • My changes do not introduce new warnings

@dannypbroadbent dannypbroadbent force-pushed the feature/order-assistants-by-supported-ids-first branch 3 times, most recently from 9b8e7a7 to f45c0e4 Compare October 30, 2024 22:00
@dannypbroadbent dannypbroadbent force-pushed the feature/order-assistants-by-supported-ids-first branch from f45c0e4 to dcb22a0 Compare November 5, 2024 09:33
@dannypbroadbent dannypbroadbent force-pushed the feature/order-assistants-by-supported-ids-first branch from dcb22a0 to a278bb9 Compare November 14, 2024 19:23
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