Refactor PhabBugz extension code to use new User.pm module for better type checking

RESOLVED FIXED

Status

()

RESOLVED FIXED
a year ago
10 months ago

People

(Reporter: dkl, Assigned: dkl)

Tracking

Production
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

45 bytes, text/x-github-pull-request
Details | Review | Splinter Review
(Assignee)

Description

a year ago
Currently Util.pm and other places of the code use utility functions to get information from Phabricator about users. This has been abstracted into a class in bug 1437383. This bug is about changing the other files to use the new User.pm class instead.
(Assignee)

Comment 1

a year ago
This should not block 1436804 for now. We can revisit this in a future story.
No longer blocks: 1436804
We'll try this as part of the next story.  Not critical but might be parallelizable with other work.
Blocks: 1429588
(Assignee)

Updated

a year ago
No longer blocks: 1429588
(Assignee)

Updated

11 months ago
Blocks: 1453733
(Assignee)

Comment 3

11 months ago
Posted file github pr #562 (obsolete) —
(Assignee)

Updated

11 months ago
Status: ASSIGNED → RESOLVED
Last Resolved: 11 months ago
Resolution: --- → FIXED

Updated

11 months ago
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 5

11 months ago
Posted file github pr #573
Attachment #8972962 - Attachment is obsolete: true
(Assignee)

Comment 6

11 months ago
New PR created for review with fixes:
https://github.com/mozilla-bteam/bmo/pull/573
(Assignee)

Updated

10 months ago
Status: REOPENED → RESOLVED
Last Resolved: 11 months ago10 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.