https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
Julian Maurice changed:
What|Removed |Added
Status|In Discussion |RESOLVED
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
--- Comment #16 from Katrin Fischer ---
What's missing here to move it out of discussion?
--
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=28201
--- Comment #15 from Tomás Cohen Arazi ---
(In reply to Julian Maurice from comment #14)
> (In reply to Tomás Cohen Arazi from comment #4)
> > I like where this is going, but I think this needs some more thinking.
> >
> > In my
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
Julian Maurice changed:
What|Removed |Added
Status|Failed QA |In Discussion
--- Comment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
--- Comment #13 from Julian Maurice ---
Patch rebased on master + added the use of anonymous subroutines for
respond_to.
There is no need to test with a debugger IMO. It's pretty obvious that we pass
a hash to respond_to, and this
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
Julian Maurice changed:
What|Removed |Added
Attachment #128360|0 |1
is obsolete|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
David Cook changed:
What|Removed |Added
See Also||https://bugs.koha-community
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
--- Comment #11 from David Cook ---
I'll open a separate report for that so I stop distracting from the real
purpose of this report.
--
You are receiving this mail because:
You are watching all bug changes.
You are the assignee
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
--- Comment #10 from David Cook ---
(In reply to Tomás Cohen Arazi from comment #9)
> (In reply to David Cook from comment #8)
> > Looking at
> > https://docs.mojolicious.org/Mojolicious/Plugin/DefaultHelpers#respond_to
> > and it
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
--- Comment #9 from Tomás Cohen Arazi ---
(In reply to David Cook from comment #8)
> Looking at
> https://docs.mojolicious.org/Mojolicious/Plugin/DefaultHelpers#respond_to
> and it looks to me like Koha/REST/V1/Biblios.pm might
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
--- Comment #8 from David Cook ---
Looking at
https://docs.mojolicious.org/Mojolicious/Plugin/DefaultHelpers#respond_to and
it looks to me like Koha/REST/V1/Biblios.pm might actually be a bit
inefficient. It looks like it runs every
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
--- Comment #7 from David Cook ---
(In reply to Tomás Cohen Arazi from comment #6)
> (In reply to Julian Maurice from comment #5)
> > (In reply to Tomás Cohen Arazi from comment #4)
> > > I like where this is going, but I think this
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
--- Comment #6 from Tomás Cohen Arazi ---
(In reply to Julian Maurice from comment #5)
> (In reply to Tomás Cohen Arazi from comment #4)
> > I like where this is going, but I think this needs some more thinking.
> >
> > In my
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
--- Comment #5 from Julian Maurice ---
(In reply to Tomás Cohen Arazi from comment #4)
> I like where this is going, but I think this needs some more thinking.
>
> In my opinion:
> - The request body should include the raw record
>
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
Tomás Cohen Arazi changed:
What|Removed |Added
Status|Needs Signoff |Failed QA
--- Comment
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
David Cook changed:
What|Removed |Added
CC||dc...@prosentient.com.au
--
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28201
Julian Maurice changed:
What|Removed |Added
Status|NEW |Needs Signoff
17 matches
Mail list logo
