[Koha-bugs] [Bug 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #33 from Fridolin Somers --- (In reply to Fridolin Somers from comment #32) > Depends on Bug 33169 not in 23.05.x In the end Bug 33169 is backported. But since this is minor I propose to keep it that way. -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Katrin Fischer changed: What|Removed |Added Status|Pushed to master|RESOLVED Resolution|--- |FIXED -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Fridolin Somers changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m --- Comment #32 from Fridolin Somers --- Depends on Bug 33169 not in 23.05.x -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #31 from Tomás Cohen Arazi --- Pushed to master for 23.11. Nice work everyone, thanks! -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Tomás Cohen Arazi changed: What|Removed |Added Version(s)||23.11.00 released in|| Status|Passed QA |Pushed to master -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Pedro Amorim changed: What|Removed |Added Attachment #154214|0 |1 is obsolete|| --- Comment #30 from Pedro Amorim --- Created attachment 155102 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155102=edit Bug 34418: Replace dynamic parameter on unnamed route node Signed-off-by: Jonathan Druart Signed-off-by: Jonathan Druart Signed-off-by: Pedro Amorim -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Pedro Amorim changed: What|Removed |Added Attachment #154213|0 |1 is obsolete|| --- Comment #29 from Pedro Amorim --- Created attachment 155101 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155101=edit Bug 34418: DO NOT PUSH - update example Signed-off-by: Jonathan Druart Signed-off-by: Pedro Amorim -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Pedro Amorim changed: What|Removed |Added Attachment #154212|0 |1 is obsolete|| --- Comment #28 from Pedro Amorim --- Created attachment 155100 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155100=edit Bug 34418: Add is_empty flag Signed-off-by: Jonathan Druart Signed-off-by: Pedro Amorim -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Pedro Amorim changed: What|Removed |Added Attachment #154211|0 |1 is obsolete|| --- Comment #27 from Pedro Amorim --- Created attachment 155099 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155099=edit Bug 34418: Allow empty nodes in breadcrumb's elements Coming from an adaptation of "Bug 33169: DO NOT PUSH Allow breadcrumbs to accept dynamic values" Signed-off-by: Jonathan Druart Signed-off-by: Pedro Amorim -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Pedro Amorim changed: What|Removed |Added Attachment #154210|0 |1 is obsolete|| --- Comment #26 from Pedro Amorim --- Created attachment 155098 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155098=edit Bug 34418: DO NOT PUSH - example Try the other patch with this patch applied. Go to http://localhost:8081/cgi-bin/koha/erm/agreements/1/items/add Without the patch you get: E-resource management > Agreements > Show agreement > > Add item to train With this patch applied: E-resource management > Agreements > Show agreement > Add item to train Signed-off-by: Jonathan Druart Signed-off-by: Pedro Amorim -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Pedro Amorim 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #25 from Jonathan Druart --- (In reply to Jonathan Druart from comment #24) > Paul, we need you here. Sorry, wrong bug, it was 34417! -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added CC||paul.dersch...@lmscloud.de --- Comment #24 from Jonathan Druart --- Paul, we need you here. -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #23 from Jonathan Druart --- Created attachment 154214 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154214=edit Bug 34418: Replace dynamic parameter on unnamed route node Signed-off-by: Jonathan Druart Signed-off-by: Jonathan Druart -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #22 from Jonathan Druart --- Created attachment 154213 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154213=edit Bug 34418: DO NOT PUSH - update example Signed-off-by: Jonathan Druart -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #21 from Jonathan Druart --- Created attachment 154212 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154212=edit Bug 34418: Add is_empty flag Signed-off-by: Jonathan Druart -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #20 from Jonathan Druart --- Created attachment 154211 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154211=edit Bug 34418: Allow empty nodes in breadcrumb's elements Coming from an adaptation of "Bug 33169: DO NOT PUSH Allow breadcrumbs to accept dynamic values" Signed-off-by: Jonathan Druart -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added Attachment #153910|0 |1 is obsolete|| Attachment #153911|0 |1 is obsolete|| Attachment #153912|0 |1 is obsolete|| Attachment #153913|0 |1 is obsolete|| Attachment #154095|0 |1 is obsolete|| --- Comment #19 from Jonathan Druart --- Created attachment 154210 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154210=edit Bug 34418: DO NOT PUSH - example Try the other patch with this patch applied. Go to http://localhost:8081/cgi-bin/koha/erm/agreements/1/items/add Without the patch you get: E-resource management > Agreements > Show agreement > > Add item to train With this patch applied: E-resource management > Agreements > Show agreement > Add item to train Signed-off-by: Jonathan Druart -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added Status|Needs Signoff |Signed Off -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added Assignee|jonathan.druart+koha@gmail. |matt.blenkinsop@ptfs-europe |com |.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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added Blocks||30708 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30708 [Bug 30708] Creation of a new 'Preservation' module -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #18 from Jonathan Druart --- (In reply to Agustín Moyano from comment #16) > I see that you added 'name: "TrainsShow"' to the node, I believe that if you > remove that name, you will see the difference with or without the patch Ok, I see now. It's working great! For instance, when on: Preservation > Trains > Show train > Add item to train "Show train" is "preservation/trains/42" with this patch Without this patch is was "preservation/trains/:train_id". -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added Attachment #153983|0 |1 is obsolete|| --- Comment #17 from Jonathan Druart --- Created attachment 154095 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154095=edit Bug 34418: Replace dynamic parameter on unnamed route node Signed-off-by: Jonathan Druart -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #16 from Agustín Moyano --- I see that you added 'name: "TrainsShow"' to the node, I believe that if you remove that name, you will see the difference with or without the patch -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #15 from Agustín Moyano --- (In reply to Jonathan Druart from comment #14) > (In reply to Agustín Moyano from comment #13) > > The last patch I pushed is to fix the link when it has parameters, for > > example, the "/preservation/trains/:train_id" on the Show train breadcrumb > > I am not sure to understand what this patch is trying to fix, could you > provide more detail please? > > Does it apply to the ERM module as well? > > I have tried on the wip for the preservation module, and: > > Preservation > Trains > Show train > Add item to train > I have "Show train" with /cgi-bin/koha/preservation/trains/1 > > With and without this last patch. > > I have adjusted the remote branch preservation_module-wip-33169 According to vue-router docs, "params are ignored if a path is provided, which is not the case for query, as shown in the example above. Instead, you need to provide the name of the route or manually specify the whole path with any parameter" That means that when we build the breadcrumb if in your routes tree you did not specify a name, NavigationItem will render the second choice Where item.path would be /preservation/trains/:train_id I did not check that branch, but if you added names to your router (as ERM has) this patch would have no effect -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #14 from Jonathan Druart --- (In reply to Agustín Moyano from comment #13) > The last patch I pushed is to fix the link when it has parameters, for > example, the "/preservation/trains/:train_id" on the Show train breadcrumb I am not sure to understand what this patch is trying to fix, could you provide more detail please? Does it apply to the ERM module as well? I have tried on the wip for the preservation module, and: Preservation > Trains > Show train > Add item to train I have "Show train" with /cgi-bin/koha/preservation/trains/1 With and without this last patch. I have adjusted the remote branch preservation_module-wip-33169 -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #13 from Agustín Moyano --- The last patch I pushed is to fix the link when it has parameters, for example, the "/preservation/trains/:train_id" on the Show train breadcrumb -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #12 from Agustín Moyano --- Created attachment 153983 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153983=edit Bug 34418: Replace dynamic parameter on unnamed route node -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added CC||agustinmoy...@theke.io --- Comment #11 from Jonathan Druart --- *** Bug 34425 has been marked as a duplicate of this bug. *** -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added Status|In Discussion |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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #10 from Matt Blenkinsop --- (In reply to Jonathan Druart from comment #3) > Matt, I am wondering if we should not have a flag in the route definition > instead, we already have is_base, is_navigation_item, is_end_node and > is_default. > > Shouldn't we have another one to flag this kind of node? Also it will > clearly show in the routes that this node is always empty and can permit to > catch bug more easily (Seeing the 2 "> >" in case it's empty but should not). This is definitely cleaner! -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added Attachment #153905|0 |1 is obsolete|| --- Comment #9 from Jonathan Druart --- Created attachment 153913 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153913=edit Bug 34418: DO NOT PUSH - update example -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added Attachment #153904|0 |1 is obsolete|| --- Comment #8 from Jonathan Druart --- Created attachment 153912 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153912=edit Bug 34418: Add is_empty flag -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added Attachment #153903|0 |1 is obsolete|| --- Comment #7 from Jonathan Druart --- Created attachment 153911 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153911=edit Bug 34418: Allow empty nodes in breadcrumb's elements Coming from an adaptation of "Bug 33169: DO NOT PUSH Allow breadcrumbs to accept dynamic values" -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added Attachment #153902|0 |1 is obsolete|| --- Comment #6 from Jonathan Druart --- Created attachment 153910 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153910=edit Bug 34418: DO NOT PUSH - example Try the other patch with this patch applied. Go to http://localhost:8081/cgi-bin/koha/erm/agreements/1/items/add Without the patch you get: E-resource management > Agreements > Show agreement > > Add item to train With this patch applied: E-resource management > Agreements > Show agreement > Add item to train -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 Jonathan Druart changed: What|Removed |Added Status|NEW |In Discussion -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #5 from Jonathan Druart --- Created attachment 153905 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153905=edit Bug 34418: DO NOT PUSH - update example -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #4 from Jonathan Druart --- Created attachment 153904 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153904=edit Bug 34418: Add is_empty flag -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #3 from Jonathan Druart --- Matt, I am wondering if we should not have a flag in the route definition instead, we already have is_base, is_navigation_item, is_end_node and is_default. Shouldn't we have another one to flag this kind of node? Also it will clearly show in the routes that this node is always empty and can permit to catch bug more easily (Seeing the 2 "> >" in case it's empty but should not). -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #2 from Jonathan Druart --- Created attachment 153903 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153903=edit Bug 34418: Allow empty nodes in breadcrumb's elements Coming from an adaptation of "Bug 33169: DO NOT PUSH Allow breadcrumbs to accept dynamic values" -- 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 34418] Allow empty nodes in breadcrumb's elements
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34418 --- Comment #1 from Jonathan Druart --- Created attachment 153902 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153902=edit Bug 34418: DO NOT PUSH - example Try the other patch with this patch applied. Go to http://localhost:8081/cgi-bin/koha/erm/agreements/1/items/add Without the patch you get: E-resource management > Agreements > Show agreement > > Add item to train With this patch applied: E-resource management > Agreements > Show agreement > Add item to train -- 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/