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

Allowing autocompletion on JComboBoxes for study cards and subjects #2564

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from

Conversation

LaurentPV
Copy link
Contributor

Allowing auto completion on study cards and existing subjects JComboBoxes.
It is not possible to edit the JComboBox with a non existing value.

It simplifies the ImportDialog.

@LaurentPV LaurentPV requested a review from michaelkain January 6, 2025 14:43
@michaelkain
Copy link
Contributor

Hi @LaurentPV, thank you for this great and simplifying PR!
Code review is fine. I have only one point during my tests, that is not optimal:
for the study-cards, normally the users filter by using the device serial number, that is at the end of the study card name. They read it from above and filter by the number. I do not think, that this works like before with your change on study cards filter?

@michaelkain michaelkain assigned LaurentPV and unassigned michaelkain Jan 7, 2025
@michaelkain
Copy link
Contributor

Hi @LaurentPV, thank you for your PR.
Thx, you have corrected a "bug" I introduced within Subject.toString().
I was only thinking of having a better log and not of the side effect on the ImportDialog combobox.

@michaelkain
Copy link
Contributor

Hi, as soon as the filtering on studycards works with device serial number, I will merge it.
With kind regards, Michael

@LaurentPV
Copy link
Contributor Author

@michaelkain dev finished, ready to be tested

@michaelkain
Copy link
Contributor

Hi Laurent! Much better user interface now, great.
I found one issue during my tests, when I clean the study card combo box, see screenshot, I can click on Start auto-import and nothing happens. Should we gray the button?
Screenshot from 2025-01-17 10-42-04

@michaelkain
Copy link
Contributor

and I have this in the logs java.lang.ClassCastException: class java.lang.String cannot be cast to class org.shanoir.uploader.model.rest.StudyCard (java.lang.String is in module java.base of loader 'bootstrap'; org.shanoir.uploader.model.rest.StudyCard is in unnamed module of loader 'app')
at org.shanoir.uploader.action.ImportFinishActionListener.actionPerformed(ImportFinishActionListener.java:72)

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