If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[bugsy] Migrate bzapi use to rest api or bzapi wrapper

RESOLVED FIXED

Status

Webtools
Elmo
P1
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: Pike, Assigned: adrian, Mentored)

Tracking

(Blocks: 1 bug)

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [lang=js][good-first-bug])

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
We should migrate as much of our bzapi code to https://wiki.mozilla.org/BMO/REST, and migrate the rest to the https://bugzilla.mozilla.org/bzapi wrapper.

This would be a good first bug for someone, as it's just involving our client side js code.

Updated

3 years ago
Blocks: 1003236
(Assignee)

Updated

3 years ago
Assignee: nobody → adrian
(Reporter)

Comment 1

3 years ago
Good that we got this assigned. Gerv notified me that the old instance goes away November 14, so raising the priority of this bug.
Priority: -- → P1

Updated

3 years ago
Blocks: 1082028
(Assignee)

Comment 2

3 years ago
Created attachment 8510064 [details] [diff] [review]
bug-1028709-migrate-bugsy.diff

Axel, as far as grepping around took me, that's the only thing I needed to change. It looks so small that I find it suspicious though... Did I miss anything?
Attachment #8510064 - Flags: review?(l10n)
(Reporter)

Comment 3

3 years ago
Comment on attachment 8510064 [details] [diff] [review]
bug-1028709-migrate-bugsy.diff

Review of attachment 8510064 [details] [diff] [review]:
-----------------------------------------------------------------

Yeah, seems that this was all that was left. Nicely, it's also just rest api right away, no need for the legacy api wrapper in these calls.
Attachment #8510064 - Flags: review?(l10n) → review+

Comment 4

3 years ago
Commit pushed to develop at https://github.com/mozilla/elmo

https://github.com/mozilla/elmo/commit/ab14f852c92fde289b21ad1af2b48935eab59b90
Fixes bug 1028709 - Migrate bugsy to the new bugzilla REST API. r=Pike

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.