[Koha-bugs] [Bug 14749] Add API route to get top issues

2018-02-18 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749

Katrin Fischer  changed:

   What|Removed |Added

 Status|Needs Signoff   |Failed QA
 CC||katrin.fisc...@bsz-bw.de,
   ||tomasco...@gmail.com

--- Comment #10 from Katrin Fischer  ---
Hi Julian,
could you add your proposal to
https://wiki.koha-community.org/wiki/REST_api_RFCs please?

I think we'd like to change a few things like branch -> library, biblionumber
-> biblio_id.

 Also adding Tomas to stop me if I am mistaken :)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 14749] Add API route to get top issues

2018-02-06 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749

Julian Maurice  changed:

   What|Removed |Added

 Status|Patch doesn't apply |Needs Signoff

--- Comment #9 from Julian Maurice  ---
Patch rebased on master + tests

Test plan:
1) Go to http://OPAC/cgi-bin/koha/opac-topissues.pl and
http://OPAC/api/v1/topissues and compare the results. They must be the same,
given a set of parameters

Parameters matching:
"Show the top" corresponds to api parameter 'count'
"From" corresponds to api parameter 'branch'
"Limit to" corresponds to api parameter 'itemtype'
"Acquired in the last" corresponds to api parameter 'newness' (in days)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 14749] Add API route to get top issues

2018-02-06 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749

--- Comment #8 from Julian Maurice  ---
Created attachment 71249
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=71249&action=edit
Bug 14749: Add unit tests for /topissues

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 14749] Add API route to get top issues

2018-02-06 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749

Julian Maurice  changed:

   What|Removed |Added

  Attachment #42408|0   |1
is obsolete||

--- Comment #7 from Julian Maurice  ---
Created attachment 71248
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=71248&action=edit
Bug 14749: Add API route /v1/topissues

See Swagger specification for documentation

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 14749] Add API route to get top issues

2016-10-16 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749

Josef Moravec  changed:

   What|Removed |Added

 CC||josef.mora...@gmail.com
 Status|Needs Signoff   |Patch doesn't apply

--- Comment #6 from Josef Moravec  ---
Needs rebase, especially because of swagger specifiction being separate in more
json files.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 14749] Add API route to get top issues

2016-09-23 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749
Bug 14749 depends on bug 13799, which changed state.

Bug 13799 Summary: Add base for building RESTful API
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13799

   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
http://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 14749] Add API route to get top issues

2016-01-04 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749

Bernardo Gonzalez Kriegel  changed:

   What|Removed |Added

 CC||bgkrie...@gmail.com

--- Comment #5 from Bernardo Gonzalez Kriegel  ---
Please, can you provide a test plan?
Don't know what to do with this.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 14749] Add API route to get top issues

2015-09-04 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749

Julian Maurice  changed:

   What|Removed |Added

 Depends on||14788

--- Comment #4 from Julian Maurice  ---
Bug split to separate the refactoring of opac-topissues.pl (bug 14788) from the
new API route


Referenced Bugs:

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14788
[Bug 14788] Move opac-topissues.pl code into C4::Circulation
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 14749] Add API route to get top issues

2015-09-04 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749

Julian Maurice  changed:

   What|Removed |Added

  Attachment #42047|0   |1
is obsolete||

--- Comment #3 from Julian Maurice  ---
Created attachment 42408
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42408&action=edit
Bug 14749: Add API route /v1/topissues

See Swagger specification for documentation

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 14749] Add API route to get top issues

2015-09-01 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749

Zeno Tajoli  changed:

   What|Removed |Added

 CC|z.taj...@cineca.it  |

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 14749] Add API route to get top issues

2015-09-01 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749

Zeno Tajoli  changed:

   What|Removed |Added

   Patch complexity|--- |Medium patch
 CC||z.taj...@cineca.it

--- Comment #2 from Zeno Tajoli  ---
Patch complexity is 'Medium' because this change has many architectural
connections

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 14749] Add API route to get top issues

2015-08-28 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749

Julian Maurice  changed:

   What|Removed |Added

 Status|ASSIGNED|Needs Signoff
 Depends on||13799


Referenced Bugs:

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13799
[Bug 13799] Add base for building RESTful API
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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 14749] Add API route to get top issues

2015-08-28 Thread bugzilla-daemon
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14749

--- Comment #1 from Julian Maurice  ---
Created attachment 42047
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42047&action=edit
Bug 14749: Add API route /v1/topissues

See Swagger specification for documentation

Refactor opac-topissues.pl code into C4::Circulation::GetTopIssues

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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/