[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Jonathan Druart changed: What|Removed |Added Keywords||additional_work_needed --- Comment #16 from Jonathan Druart --- It broke ERM/Agreements_spec.ts and ERM/Licenses_spec.ts, please fix ASAP. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Jonathan Druart changed: What|Removed |Added Status|Pushed to main |Passed QA Keywords||additional_work_needed --- Comment #13 from Jonathan Druart --- (In reply to Pedro Amorim from comment #11) > Created attachment 178507 [details] > filters out of sync > > This has caused the filters to become out of sync in agreements, and likely > licenses too, haven't tested. Look at picture, the values for the > "Description" filter are the "Status" ones, etc. > > filters_options inside tableOptions needs to be adjusted accordingly. Last patch fixes this. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 --- Comment #12 from Jonathan Druart --- Created attachment 179119 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179119&action=edit Bug 37273: Fix offset for column filters -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Katrin Fischer changed: What|Removed |Added Keywords|additional_work_needed | --- Comment #15 from Katrin Fischer --- Follow-up pushed! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Katrin Fischer changed: What|Removed |Added Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 --- Comment #14 from Katrin Fischer --- Pushed for 25.05! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 --- Comment #11 from Pedro Amorim --- Created attachment 178507 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178507&action=edit filters out of sync This has caused the filters to become out of sync in agreements, and likely licenses too, haven't tested. Look at picture, the values for the "Description" filter are the "Status" ones, etc. filters_options inside tableOptions needs to be adjusted accordingly. This work is adding more niche specific behavior to specific tables, it's bad practice. This should be built and considered on top of bug 38201. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 --- Comment #10 from Katrin Fischer --- Pushed for 25.05! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Katrin Fischer changed: What|Removed |Added Status|Passed QA |Pushed to main Version(s)||25.05.00 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Katrin Fischer changed: What|Removed |Added Keywords|Academy | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Lucas Gass (lukeg) changed: What|Removed |Added Attachment #172137|0 |1 is obsolete|| --- Comment #9 from Lucas Gass (lukeg) --- Created attachment 176741 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176741&action=edit Bug 37273: (follow-up) Remove parentheses and # from column The new ID column shows numbers like "(#2)". I think the parentheses and hash symbol are unnecessary and inconsistent with the way we do it in other places and should be removed. To test, apply the patch and rebuild the staff interface CSS: Either run 'yarn build' from the command line or if you're using a sandbox, click the "Actions" menu for your sandbox and choose "Build CSS and JS." Follow the previous test plan and confirm that the ID column looks correct. Signed-off-by: Lucas Gass -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Lucas Gass (lukeg) changed: What|Removed |Added Attachment #169107|0 |1 is obsolete|| --- Comment #8 from Lucas Gass (lukeg) --- Created attachment 176740 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176740&action=edit Bug 37273: Added ID column to ERM Agreements and Licenses Tables To test 1. Apply patch, yarn build, restart_all 2. System preferences -> ERMModule -> Enable 3. E-resource management -> Agreements -> "+New Agreement" 4. Notice ID is removed from name and put in it's own column 5. Repeat with licenses table Signed-off-by: Lucas Gass -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Lucas Gass (lukeg) changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Lucas Gass (lukeg) changed: What|Removed |Added Text to go in the||The enhancement adds an ID release notes||column to the ERM's ||agreement table which is ||configurable through Table ||settings. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Lucas Gass (lukeg) changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |lu...@bywatersolutions.com |y.org | CC||lu...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 --- Comment #7 from Owen Leonard --- Created attachment 172137 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172137&action=edit Bug 37273: (follow-up) Remove parentheses and # from column The new ID column shows numbers like "(#2)". I think the parentheses and hash symbol are unnecessary and inconsistent with the way we do it in other places and should be removed. To test, apply the patch and rebuild the staff interface CSS: Either run 'yarn build' from the command line or if you're using a sandbox, click the "Actions" menu for your sandbox and choose "Build CSS and JS." Follow the previous test plan and confirm that the ID column looks correct. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 davedaghita changed: What|Removed |Added Status|Needs Signoff |Signed Off --- Comment #6 from davedaghita --- The ID# column worked. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 davedaghita changed: What|Removed |Added CC||ddagh...@mckinneytexas.org -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Michaela Sieber changed: What|Removed |Added CC||michaela.sie...@kit.edu -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273
--- Comment #5 from Pedro Amorim ---
Comment on attachment 169107
--> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=169107
Bug 37273: Added ID column to ERM Agreements and Licenses Tables
Review of attachment 169107:
-->
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=37273&attachment=169107)
-
::: koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsList.vue
@@ +268,3 @@
> {
> title: __("Name"),
> data: "me.name:me.agreement_id",
We can probably also change this to 'me.name' only.
::: koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/LicensesList.vue
@@ +161,3 @@
> {
> title: __("Name"),
> data: "me.name:me.license_id",
We can probably also change this to 'me.name' only.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Eric Garcia changed: What|Removed |Added CC||cubingguy...@gmail.com Assignee|jonathan.dru...@gmail.com |cubingguy...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 --- Comment #4 from Eric Garcia --- Created attachment 169107 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=169107&action=edit Bug 37273: Added ID column to ERM Agreements and Licenses Tables To test 1. Apply patch, yarn build, restart_all 2. System preferences -> ERMModule -> Enable 3. E-resource management -> Agreements -> "+New Agreement" 4. Notice ID is removed from name and put in it's own column 5. Repeat with licenses table -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Eric Garcia changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 --- Comment #3 from Esther Melander --- It's not clear what the number means. Labeling it makes it understood the number is an ID number. I can see some cases where a number like '#1' is part of a title on the titles table, which can be confusing. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 --- Comment #2 from Jonathan Druart --- I guess I just wanted to avoid a new column in the table. We can have it in a separate column if it is more convenient for people. However I would not hide it, in case you want to search/order by the IDs. Not sure to understand why it is considered confusing however. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 37273] Add ID column to Agreements table in the ERM
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37273 Pedro Amorim changed: What|Removed |Added Patch complexity|--- |Trivial patch Keywords||Academy --- Comment #1 from Pedro Amorim --- (In reply to Esther Melander from comment #0) > Created attachment 168606 [details] > ERM ID number appended to agreement name > > When creating an Agreement or License in the ERM, an ID number is added to > the agreement or license name. If it is necessary for the ID number to be > visible, it should be in a separate column from the name. Having the ID > number appended to the name is a little confusing (see example image). > Another option is to hide the hide ID number if it is not needed. I think this makes sense. @Joubu do we remember the initial reason why we decided the way it is currently? Trying to understand if there are any blockers for this. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
