[Koha-bugs] [Bug 35919] Add record sources CRUD

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35919

--- Comment #59 from Jonathan Druart <jonathan.druart at gmail.com> ---
(In reply to Tomás Cohen Arazi from comment #39)
> Marking NSO again, as I rewrote most of the Vue part of this dev.
> 
> Some things I'm not comfortable with, but happy to follow the current
> standard practice for now, and evolve later.
> 
> I kept the URL rewrite rule we originally submitted. I think requiring the
> trailing slash (/) if my rewrite avoids it, doesn't make sense.
> 
> I can submit a bug for redoing the rest of the rewrites this way if required
> by QA (if this applies, as I'm not familiar with the little details the
> other modules might have making them requiring this).
> 
> I also liked it more when the API client was instantiated globally instead
> on each place it is used. As it is a stateless object it should cause no
> issues. I moved it to match the current practice and will file a separate
> bug to restore the original implementation and discuss it in place.
> 
> Thanks everyone!

Did you open the bugs?

-- 
You are receiving this mail because:
You are watching all bug changes.