[Bug 40906] New: Change the "Create report from mana" interface, to make it look more like a plugin market place/app store
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Bug ID: 40906 Summary: Change the "Create report from mana" interface, to make it look more like a plugin market place/app store Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Mana-kb Assignee: koha-bugs@lists.koha-community.org Reporter: elias.lorgnier@biblibre.com QA Contact: testopia@bugs.koha-community.org CC: alex.arnaud@biblibre.com The idea is to create a better interface than search only for importing reports from mana. A marketplace would allow discovering new reports, while maintaining a good searchability. The information presented as a grid of cards would allow seeing immediately informations that are important to evaluate the relevance and "reliability" of a report : title, description, author, number of download. It would also allow for a better visual hierarchy between title, description, and other information. Each card will show the following informations - Title - Author - number of downloads - description (Note in Koha) The full SQL code from the report will be available by clicking on each card, as well as the full description of the report. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |40885 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40885 [Bug 40885] [OMNIBUS] Mana improvements to reports and subscriptions sharing -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |elias.lorgnier@biblibre.com |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 --- Comment #1 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 190148 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190148&action=edit Bug 40906: Redesign Mana report import UI Why : This patch is a revamp of the interface of the menue "Import SQL from Mana". It is supposed to make it feel more like a marketplace of plugin or apps. I believe it is more coherent visually and fonctionally with what this feature is doing. Also, reports are immediatly displayed when the modal opens, inviting the user to explore report, while maintaining the ability to search for it. When the patch 40826 will be applied, we'll be able to sort reports based on the number of imports they recieved. It also includes : - Moving the search of report from the deprecated svc to the rest API - Paginating results so it doesn't load 1300 reports each time we search for reports. - Pagination makes it future proof in case of an augmentation of available reports. Test plan : - Clone the test-bug-40906 branch from my repo of mana : https://gitlab.com/elorgnier/koha-mana/-/tree/test-bug-40906?ref_type=heads - Run docker compose up in the repo - in the .env file of ktd, for the MANA_URL option, use 127.17.0.1:5000 (if you have the default host ip defined in the docker bridge network) - run ktd - in koha > Administration, configure a mana account with random username and email (no need to validate it) - In the report section, select new SQL report, and then in the dropdown menu, New SQL from mana. By default there is no report loaded in mana, so you will want to create sample reports to be able to test it, i'll push a sample to the test branch of mana as soon as possible. You can create your own sample or mount a dump of the existing database in the docker compose of mana by mounting the sql dump in the place of the /sql/schema.sql -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #190148|0 |1 is obsolete| | --- Comment #2 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 190396 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190396&action=edit Bug 40906: Redesign Mana report import UI Why : This patch is a revamp of the interface of the menue "Import SQL from Mana". It is supposed to make it feel more like a marketplace of plugin or apps. I believe it is more coherent visually and fonctionally with what this feature is doing. Also, reports are immediatly displayed when the modal opens, inviting the user to explore report, while maintaining the ability to search for it. When the patch 40826 will be applied, we'll be able to sort reports based on the number of imports they recieved. It also includes : - Moving the search of report from the deprecated svc to the rest API - Paginating results so it doesn't load 1300 reports each time we search for reports. - Pagination makes it future proof in case of an augmentation of available reports. Test plan : - Clone the test-bug-40906 branch from my repo of mana : https://gitlab.com/elorgnier/koha-mana/-/tree/test-bug-40906?ref_type=heads - Run docker compose up in the repo - in the .env file of ktd, for the MANA_URL option, use 127.17.0.1:5000 (if you have the default host ip defined in the docker bridge network) - run ktd - in koha > Administration, configure a mana account with random username and email (no need to validate it) - In the report section, select new SQL report, and then in the dropdown menu, New SQL from mana. By default there is no report loaded in mana, so you will want to create sample reports to be able to test it, i'll push a sample to the test branch of mana as soon as possible. You can create your own sample or mount a dump of the existing database in the docker compose of mana by mounting the sql dump in the place of the /sql/schema.sql -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@openfifth.c | |o.uk -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #190396|0 |1 is obsolete| | --- Comment #3 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 190544 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190544&action=edit Bug 40906: Redesign Mana report import UI Why : This patch is a revamp of the interface of the menue "Import SQL from Mana". It is supposed to make it feel more like a marketplace of plugin or apps. I believe it is more coherent visually and fonctionally with what this feature is doing. Also, reports are immediatly displayed when the modal opens, inviting the user to explore report, while maintaining the ability to search for it. When the patch 40826 will be applied, we'll be able to sort reports based on the number of imports they recieved. It also includes : - Moving the search of report from the deprecated svc to the rest API - Paginating results so it doesn't load 1300 reports each time we search for reports. - Pagination makes it future proof in case of an augmentation of available reports. Test plan : - Clone the test-bug-40906 branch from my repo of mana : https://gitlab.com/elorgnier/koha-mana/-/tree/test-bug-40906?ref_type=heads - Run docker compose up in the repo - in the .env file of ktd, for the MANA_URL option, use 127.17.0.1:5000 (if you have the default host ip defined in the docker bridge network) - run ktd - in koha > Administration, configure a mana account with random username and email (no need to validate it) - In the report section, select new SQL report, and then in the dropdown menu, New SQL from mana. By default there is no report loaded in mana, so you will want to create sample reports to be able to test it, i'll push a sample to the test branch of mana as soon as possible. You can create your own sample or mount a dump of the existing database in the docker compose of mana by mounting the sql dump in the place of the /sql/schema.sql -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 --- Comment #4 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 190545 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190545&action=edit Bug 40906: Dropdown sort button in mana import UI Using the datatable "collection" button, i added a dropdown menu to the datatable, making it possible to sort results, so no feature are lost in the patch. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Sponsorship status|--- |Sponsored Comma delimited| |Biblibre list of Sponsors| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #190544|0 |1 is obsolete| | --- Comment #5 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 190660 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190660&action=edit Bug 40906: Redesign Mana report import UI Why : This patch is a revamp of the interface of the menue "Import SQL from Mana". It is supposed to make it feel more like a marketplace of plugin or apps. I believe it is more coherent visually and fonctionally with what this feature is doing. Also, reports are immediatly displayed when the modal opens, inviting the user to explore report, while maintaining the ability to search for it. When the patch 40826 will be applied, we'll be able to sort reports based on the number of imports they recieved. It also includes : - Moving the search of report from the deprecated svc to the rest API - Paginating results so it doesn't load 1300 reports each time we search for reports. - Pagination makes it future proof in case of an augmentation of available reports. Test plan : - Clone the mana repo on yout machine (from https://gitlab.com/koha-community/koha-mana) - Run docker compose up in the repo - in the .env file of ktd, for the MANA_URL option, use http://<gateway ip in the kokhanet bridge network>:5000 - run ktd - in koha > Administration, configure a mana account with random username and email (no need to validate it) - In the report section, select new SQL report, and then in the dropdown menu, New SQL from mana. By default there is no report loaded in mana, so you will want to create sample reports to be able to test it, i'll push a sample to the test branch of mana as soon as possible. You can create your own sample or mount a dump of the existing database in the docker compose of mana by mounting the sql dump in the place of the /sql/schema.sql -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #190660|0 |1 is obsolete| | --- Comment #6 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 190661 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190661&action=edit Bug 40906: Redesign Mana report import UI Why : This patch is a revamp of the interface of the menue "Import SQL from Mana". It is supposed to make it feel more like a marketplace of plugin or apps. I believe it is more coherent visually and fonctionally with what this feature is doing. Also, reports are immediatly displayed when the modal opens, inviting the user to explore report, while maintaining the ability to search for it. When the patch 40826 will be applied, we'll be able to sort reports based on the number of imports they recieved. It also includes : - Moving the search of report from the deprecated svc to the rest API - Paginating results so it doesn't load 1300 reports each time we search for reports. - Pagination makes it future proof in case of an augmentation of available reports. Test plan : - Clone the mana repo on yout machine (from https://gitlab.com/koha-community/koha-mana) - Run docker compose up in the repo - in the .env file of ktd, for the MANA_URL option, use http://<gateway ip in the kokhanet bridge network>:5000 - run ktd - in koha > Administration, configure a mana account with random username and email (no need to validate it) - In the report section, select new SQL report, and then in the dropdown menu, New SQL from mana. By default there is no report loaded in mana, so you will want to create sample reports to be able to test it, i'll push a sample to the test branch of mana as soon as possible. You can create your own sample or mount a dump of the existing database in the docker compose of mana by mounting the sql dump in the place of the /sql/schema.sql -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #190545|0 |1 is obsolete| | --- Comment #7 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 190662 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190662&action=edit Bug 40906: Dropdown sort button in mana import UI Using the datatable "collection" button, i added a dropdown menu to the datatable, making it possible to sort results, so no feature are lost in the patch. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 --- Comment #8 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Initial testing and this is looking great. I got the Mana Docker up and running and connected, but I am struggling to populate it with any reports. For some reason, my shares from KTD never appear on the Mana side -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #190661|0 |1 is obsolete| | --- Comment #9 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 191020 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=191020&action=edit Bug 40906: Redesign Mana report import UI Why : This patch is a revamp of the interface of the menue "Import SQL from Mana". It is supposed to make it feel more like a marketplace of plugin or apps. I believe it is more coherent visually and fonctionally with what this feature is doing. Also, reports are immediatly displayed when the modal opens, inviting the user to explore report, while maintaining the ability to search for it. When the patch 40826 will be applied, we'll be able to sort reports based on the number of imports they recieved. It also includes : - Moving the search of report from the deprecated svc to the rest API - Paginating results so it doesn't load 1300 reports each time we search for reports. - Pagination makes it future proof in case of an augmentation of available reports. Test plan : - Clone the mana repo on yout machine (from https://gitlab.com/koha-community/koha-mana) - Run docker compose up in the repo - in the .env file of ktd, for the MANA_URL option, use http://<gateway ip in the kokhanet bridge network>:5000 - run ktd - in koha > Administration, configure a mana account with random username and email (no need to validate it) - In the report section, select new SQL report, and then in the dropdown menu, New SQL from mana. By default there is no report loaded in mana, so you will want to create sample reports to be able to test it, i'll push a sample to the test branch of mana as soon as possible. You can create your own sample or mount a dump of the existing database in the docker compose of mana by mounting the sql dump in the place of the /sql/schema.sql -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #190662|0 |1 is obsolete| | --- Comment #10 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 191021 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=191021&action=edit Bug 40906: Dropdown sort button in mana import UI Using the datatable "collection" button, i added a dropdown menu to the datatable, making it possible to sort results, so no feature are lost in the patch. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |41652 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41652 [Bug 41652] Implement the relevance search order by default in the import from mana interface -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #191020|0 |1 is obsolete| | --- Comment #11 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 192018 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=192018&action=edit Bug 40906: Redesign Mana report import UI Why : This patch is a revamp of the interface of the menue "Import SQL from Mana". It is supposed to make it feel more like a marketplace of plugin or apps. I believe it is more coherent visually and fonctionally with what this feature is doing. Also, reports are immediatly displayed when the modal opens, inviting the user to explore report, while maintaining the ability to search for it. When the patch 40826 will be applied, we'll be able to sort reports based on the number of imports they recieved. It also includes : - Moving the search of report from the deprecated svc to the rest API - Paginating results so it doesn't load 1300 reports each time we search for reports. - In follow up, adds a drop down button to sort reports - Pagination makes it future proof in case of an augmentation of available reports. Test plan : - Clone the mana repo on yout machine (from https://gitlab.com/koha-community/koha-mana) - Set up the mana utility to start mana - run `mana up` to launch a mana instance (it will create a mana network with docker) - run `ktd --mana up` - in koha > Administration > ManaKB settings, set the mana usage to yes - In the report section, select new SQL report, and then in the dropdown menu, New SQL from mana. By default there is no report loaded in mana, so you will want to create sample reports to be able to test thr patch. You can simply uncomment the line in the `compose/base.yml`, under db -> volumes to add the sample reports. To be able to export your reports from koha to mana requires that you create a librarian account, wich can be done from koha administration page, and that you verify that account, which can only be done directly with SQL or by allowing mana to send emails for now, wich is not configurable easely. If you already have a mana instance running, you can use the --mana-url option of ktd to choose the url of your mana instance. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #191021|0 |1 is obsolete| | --- Comment #12 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 192019 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=192019&action=edit Bug 40906: Dropdown sort button in mana import UI Using the datatable "collection" button, i added a dropdown menu to the datatable, making it possible to sort results, so no feature are lost in the patch. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #192018|0 |1 is obsolete| | --- Comment #13 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 192098 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=192098&action=edit Bug 40906: Redesign Mana report import UI Why : This patch is a revamp of the interface of the menue "Import SQL from Mana". It is supposed to make it feel more like a marketplace of plugin or apps. I believe it is more coherent visually and fonctionally with what this feature is doing. Also, reports are immediatly displayed when the modal opens, inviting the user to explore report, while maintaining the ability to search for it. When the patch 40826 will be applied, we'll be able to sort reports based on the number of imports they recieved. It also includes : - Moving the search of report from the deprecated svc to the rest API - Paginating results so it doesn't load 1300 reports each time we search for reports. - In follow up, adds a drop down button to sort reports - Pagination makes it future proof in case of an augmentation of available reports. Test plan : - Clone the mana repo on your machine (from https://gitlab.com/koha-community/koha-mana) - By default there is no report loaded in mana, so you will want to create sample reports to be able to test the patch. You can simply uncomment the line in the `compose/base.yml`, under db -> volumes to add the sample reports. - Add the bin/mana utility to your path - run `mana up` to launch mana and create a docker network koha will connect to (see how to use the mana utility here : https://gitlab.com/koha-community/koha-mana/-/tree/main/compose?ref_type=hea...) - run `ktd --mana up` to connect koha to mana - in koha > Administration > ManaKB settings, enable Mana (no need to create a secority token to test this patch) - In the report section, select new SQL report, and then in the dropdown menu, New SQL from mana. To be able to export your reports from koha to mana requires that you create a librarian account, wich can be done from koha administration page, and that you verify that account, which can only be done directly with SQL or by allowing mana to send emails for now, wich is not configurable easely. If you already have a mana instance running, you can use the --mana-url option of ktd to choose the url of your mana instance. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #192019|0 |1 is obsolete| | --- Comment #14 from elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> --- Created attachment 192099 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=192099&action=edit Bug 40906: Dropdown sort button in mana import UI Using the datatable "collection" button, i added a dropdown menu to the datatable, making it possible to sort results, so no feature are lost in the patch. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Change the "Create report |Update the "New SQL from |from mana" interface, to |mana" interface |make it look more like a | |plugin market place/app | |store | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 elias.lorgnier@biblibre.com <elias.lorgnier@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Lisette Scheer <lisette@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lisette@bywatersolutions.co | |m -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |arthur.suzuki@biblibre.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Michelle Spinney <mspinney@clamsnet.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mspinney@clamsnet.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Elias <elias.lorgnier@girofle.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #192098|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Elias <elias.lorgnier@girofle.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #192099|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 --- Comment #15 from Elias <elias.lorgnier@girofle.org> --- Created attachment 196272 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196272&action=edit Bug 40906: Redesign Mana report import UI Why : This patch is a revamp of the interface of the menue "Import SQL from Mana". It is supposed to make it feel more like a marketplace of plugin or apps. I believe it is more coherent visually and fonctionally with what this feature is doing. Also, reports are immediatly displayed when the modal opens, inviting the user to explore report, while maintaining the ability to search for it. When the patch 40826 will be applied, we'll be able to sort reports based on the number of imports they recieved. It also includes : - Moving the search of report from the deprecated svc to the rest API - Paginating results so it doesn't load 1300 reports each time we search for reports. - In follow up, adds a drop down button to sort reports - Pagination makes it future proof in case of an augmentation of available reports. Test plan : - Clone the mana repo on your machine (from https://gitlab.com/koha-community/koha-mana) - By default there is no report loaded in mana, so you will want to create sample reports to be able to test the patch. You can simply uncomment the line in the `compose/base.yml`, under db -> volumes to add the sample reports. - Add the bin/mana utility to your path - run `mana up` to launch mana and create a docker network koha will connect to (see how to use the mana utility here : https://gitlab.com/koha-community/koha-mana/-/tree/main/compose?ref_type=hea...) - run `ktd --mana up` to connect koha to mana - in koha > Administration > ManaKB settings, enable Mana (no need to create a secority token to test this patch) - In the report section, select new SQL report, and then in the dropdown menu, New SQL from mana. To be able to export your reports from koha to mana requires that you create a librarian account, wich can be done from koha administration page, and that you verify that account, which can only be done directly with SQL or by allowing mana to send emails for now, wich is not configurable easely. If you already have a mana instance running, you can use the --mana-url option of ktd to choose the url of your mana instance. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 --- Comment #16 from Elias <elias.lorgnier@girofle.org> --- Created attachment 196273 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196273&action=edit Bug 40906: Dropdown sort button in mana import UI Using the datatable "collection" button, i added a dropdown menu to the datatable, making it possible to sort results, so no feature are lost in the patch. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Emmanuel Bétemps <e.betemps@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196272|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Emmanuel Bétemps <e.betemps@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196273|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 --- Comment #17 from Emmanuel Bétemps <e.betemps@gmail.com> --- Created attachment 196341 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196341&action=edit Bug 40906: Redesign Mana report import UI Why : This patch is a revamp of the interface of the menue "Import SQL from Mana". It is supposed to make it feel more like a marketplace of plugin or apps. I believe it is more coherent visually and fonctionally with what this feature is doing. Also, reports are immediatly displayed when the modal opens, inviting the user to explore report, while maintaining the ability to search for it. When the patch 40826 will be applied, we'll be able to sort reports based on the number of imports they recieved. It also includes : - Moving the search of report from the deprecated svc to the rest API - Paginating results so it doesn't load 1300 reports each time we search for reports. - In follow up, adds a drop down button to sort reports - Pagination makes it future proof in case of an augmentation of available reports. Test plan : - Clone the mana repo on your machine (from https://gitlab.com/koha-community/koha-mana) - By default there is no report loaded in mana, so you will want to create sample reports to be able to test the patch. You can simply uncomment the line in the `compose/base.yml`, under db -> volumes to add the sample reports. - Add the bin/mana utility to your path - run `mana up` to launch mana and create a docker network koha will connect to (see how to use the mana utility here : https://gitlab.com/koha-community/koha-mana/-/tree/main/compose?ref_type=hea...) - run `ktd --mana up` to connect koha to mana - in koha > Administration > ManaKB settings, enable Mana (no need to create a secority token to test this patch) - In the report section, select new SQL report, and then in the dropdown menu, New SQL from mana. To be able to export your reports from koha to mana requires that you create a librarian account, wich can be done from koha administration page, and that you verify that account, which can only be done directly with SQL or by allowing mana to send emails for now, wich is not configurable easely. If you already have a mana instance running, you can use the --mana-url option of ktd to choose the url of your mana instance. Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 --- Comment #18 from Emmanuel Bétemps <e.betemps@gmail.com> --- Created attachment 196342 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196342&action=edit Bug 40906: Dropdown sort button in mana import UI Using the datatable "collection" button, i added a dropdown menu to the datatable, making it possible to sort results, so no feature are lost in the patch. Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Emmanuel Bétemps <e.betemps@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |e.betemps@gmail.com Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 --- Comment #19 from Thomas Klausner <domm@plix.at> --- Created attachment 196346 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196346&action=edit Bug 40906: (QA-followup) tidy template, add missing newline Patch from commit 6aa4df4 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Thomas Klausner <domm@plix.at> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #20 from Thomas Klausner <domm@plix.at> --- Worked as described in the test plan, also looks very nice :-) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Thomas Klausner <domm@plix.at> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196341|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Thomas Klausner <domm@plix.at> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196342|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Thomas Klausner <domm@plix.at> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196346|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 --- Comment #21 from Thomas Klausner <domm@plix.at> --- Created attachment 196355 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196355&action=edit Bug 40906: Redesign Mana report import UI Why : This patch is a revamp of the interface of the menue "Import SQL from Mana". It is supposed to make it feel more like a marketplace of plugin or apps. I believe it is more coherent visually and fonctionally with what this feature is doing. Also, reports are immediatly displayed when the modal opens, inviting the user to explore report, while maintaining the ability to search for it. When the patch 40826 will be applied, we'll be able to sort reports based on the number of imports they recieved. It also includes : - Moving the search of report from the deprecated svc to the rest API - Paginating results so it doesn't load 1300 reports each time we search for reports. - In follow up, adds a drop down button to sort reports - Pagination makes it future proof in case of an augmentation of available reports. Test plan : - Clone the mana repo on your machine (from https://gitlab.com/koha-community/koha-mana) - By default there is no report loaded in mana, so you will want to create sample reports to be able to test the patch. You can simply uncomment the line in the `compose/base.yml`, under db -> volumes to add the sample reports. - Add the bin/mana utility to your path - run `mana up` to launch mana and create a docker network koha will connect to (see how to use the mana utility here : https://gitlab.com/koha-community/koha-mana/-/tree/main/compose?ref_type=hea...) - run `ktd --mana up` to connect koha to mana - in koha > Administration > ManaKB settings, enable Mana (no need to create a secority token to test this patch) - In the report section, select new SQL report, and then in the dropdown menu, New SQL from mana. To be able to export your reports from koha to mana requires that you create a librarian account, wich can be done from koha administration page, and that you verify that account, which can only be done directly with SQL or by allowing mana to send emails for now, wich is not configurable easely. If you already have a mana instance running, you can use the --mana-url option of ktd to choose the url of your mana instance. Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com> Signed-off-by: Thomas Klausner <domm@plix.at> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 --- Comment #22 from Thomas Klausner <domm@plix.at> --- Created attachment 196356 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196356&action=edit Bug 40906: Dropdown sort button in mana import UI Using the datatable "collection" button, i added a dropdown menu to the datatable, making it possible to sort results, so no feature are lost in the patch. Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com> Signed-off-by: Thomas Klausner <domm@plix.at> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 --- Comment #23 from Thomas Klausner <domm@plix.at> --- Created attachment 196357 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196357&action=edit Bug 40906: (QA-followup) tidy template, add missing newline Signed-off-by: Thomas Klausner <domm@plix.at> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |A marketplace that allows release notes| |discovering new reports, | |while maintaining a good | |searchability. The | |information presented as a | |grid of cards would allow | |seeing immediately | |informations that are | |important to evaluate the | |relevance and "reliability" | |of a report : title, | |description, author, number | |of download. It would also | |allow for a better visual | |hierarchy between title, | |description, and other | |information. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |domm@plix.at |y.org | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40906 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA CC| |lucas@bywatersolutions.com --- Comment #24 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- With this patchset applied: kohadev-koha@kohadevbox:koha(main)$ prove xt/api.t xt/api.t .. 3/8 # Failed test 'No tag errors in the spec' # at xt/api.t line 130. # Structures begin differing at: # $got->[0] = 'get /mana/reports -> uses tag 'mana' not present in top level list' # $expected->[0] = Does not exist get /mana/reports -> uses tag 'mana' not present in top level list get /mana/reports -> uses tag 'reports' not present in top level list # Looks like you failed 1 test of 1. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org