Skip to content

Conversation

@jmbrunskill
Copy link
Contributor

@jmbrunskill jmbrunskill commented Nov 13, 2025

Fixes #9762

πŸ‘©πŸ»β€πŸ’» What does this PR do?

Allows you to search for prefs!

image

πŸ’Œ Any notes for the reviewer?

AI Generated code, but looks fine? UI Could probably be improved and more consistent but seems like a helpful improvement for no effort?

Haven't added to Global prefs but looks like an easy change...

πŸ§ͺ Testing

  • (e.g.) Central Sync server with 1 Legacy Desktop remote site and 1 OMS remote site running this PR
  • (e.g.) This sample datafile: google drive link
  • (e.g.) Open a requisition with some lines
  • (e.g.) Make a couple invoices supplying some amount of those lines
  • (e.g.) Review that "issued" column is the sum of the amount already issued in invoices for this requisition

πŸ“ƒ Documentation

  • Part of an epic: documentation will be completed for the feature as a whole
  • No documentation required: no user facing changes or a bug fix which isn't a change in behaviour
  • These areas should be updated or checked:
    1.
    2.

πŸ“ƒ Reviewer Checklist

The PR Reviewer(s) should fill out this section before approving the PR

Breaking Changes

  • No Breaking Changes in the Graphql API
  • Technically some Breaking Changes but not expected to impact any integrations

Issue Review

  • All requirements in original issue have been covered
  • A follow up issue(s) have been created to cover additional requirements

Tests Pass

  • Postgres
  • SQLite
  • Frontend

@github-actions github-actions bot added this to the v2.14.0 milestone Nov 13, 2025
@github-actions github-actions bot added enhancement New feature or request Team Whio James, Chris, Zac, Carl & Ravi labels Nov 13, 2025
@jmbrunskill jmbrunskill changed the title 9696 prefsearch Search for the pref you want! Nov 13, 2025
@mark-prins
Copy link
Collaborator

😍 great idea! thanks @jmbrunskill that's super helpful!!

can we please pause merging until I can get a design opinion - I don't like that the search looks like a preference itself! I think it should stand out more; would like a second opinion.

and the issue linked isn't specifically for this; we shouldn't close 9696 when this PR is merged, and there's potential to be misled. can you create an issue for the pref search and link to this PR please

@jmbrunskill
Copy link
Contributor Author

😍 great idea! thanks @jmbrunskill that's super helpful!!

can we please pause merging until I can get a design opinion - I don't like that the search looks like a preference itself! I think it should stand out more; would like a second opinion.

and the issue linked isn't specifically for this; we shouldn't close 9696 when this PR is merged, and there's potential to be misled. can you create an issue for the pref search and link to this PR please

Yeah I think the UI could be improved but glad you like the concept. Will raise an issue. Needs to be done for the Global Prefs too.

I think we should also consider this ASAP too...
#7802

There two prefs are in dire need to explantion for example
image

@github-actions github-actions bot removed this from the v2.14.0 milestone Nov 13, 2025
@jmbrunskill jmbrunskill changed the base branch from develop to 9696-warn-stocktake November 14, 2025 03:03
Base automatically changed from 9696-warn-stocktake to develop November 17, 2025 03:09
@mark-prins
Copy link
Collaborator

some designs πŸ˜„

please ignore the slightly different look of the OK button and toggles - the wrong MUI component has been used in figma:

telegram-cloud-photo-size-1-4963093302378957589-y

telegram-cloud-photo-size-1-4963093302378957590-y

points to note

  • full width filter bar (am leaning into the filter rather than search, bc the action will filter the list of prefs shown)
  • a 'start icon' of the filter
  • and a clear option on the input
  • left align the text in the input

and to consider:

  • we didn't include the word 'filter' as a header or placeholder. I thought it would be evident enough
  • Ideally the top section of this modal would have a light grey background, as per the requisition detail modal, to prevent the tab control floating in the middle of a whitespace wonderland. could we do that with this PR? please....??

@roxy-dao roxy-dao changed the base branch from develop to v2.14.0-RC November 20, 2025 10:30
@github-actions github-actions bot removed this from the Christmas Release milestone Nov 20, 2025
@mark-prins
Copy link
Collaborator

thanks @jmbrunskill
have updated to look like this:

Screenshot 2025-11-26 at 12 42 27β€―PM Screenshot 2025-11-26 at 12 42 34β€―PM Screenshot 2025-11-26 at 12 20 24β€―PM

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

Labels

enhancement New feature or request Team Whio James, Chris, Zac, Carl & Ravi

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Preference Search

3 participants