Age | Commit message (Collapse) | Author | Files | Lines | |
---|---|---|---|---|---|
2018-04-11 | Bug 1453122 - The phabbugz feed daemon should use IO::Async::Timer instead ↵ | dklawren | 2 | -13/+55 | |
of using while loop | |||||
2018-03-29 | Bug 1441063: Fix the unaccepted revision comment (PhabBugz) | Piotr Zalewa | 1 | -1/+7 | |
* PhabBugz: Fix the unaccepted revision comment Summary: Currently, we're sending the "User removed from revision" comment when the Accept flag has been removed from a revision. This could happen in a number of use cases: 1. The reviewer resigned from being a reviewer. 2. The reviewer has been removed from reviewers list. 3. Someone (author, reviewer) removed acceptance of the revision by changing its status to "Needs Review". This patch is sending a "flag is deactivated" or "reviewer removed from revision" depending on the current status of the reviewer. Test Plan: Accept a revision. Change the revision to "Needs Review". Check the bug comment. Accept the revision. Remove the user from reviewers list. Check the bug comment. Reviewers: dkl Bug #: 1441063 Differential Revision: https://phabricator.services.mozilla.com/D809 * Styling fixed as requested in review. * Perl style fixes | |||||
2018-03-27 | Bug 1440829 - Bugzilla comment for Phabricator commit should include entire ↵ | dklawren | 3 | -7/+9 | |
commit message, not just first line | |||||
2018-03-23 | Bug 1446974 - Revert changes of the subscribers list on secure revisions | dklawren | 1 | -3/+5 | |
2018-03-20 | Bug 1447291 - Remove Apache2::Log from PhabBugs/Push in favor of logging ↵ | Dylan William Hardison | 4 | -82/+41 | |
framework | |||||
2018-03-06 | Bug 1443537 - Add utility functions for switching database handle to ↵ | Dylan William Hardison | 1 | -12/+7 | |
writeable or read-only mode | |||||
2018-03-02 | Bug 1402494 - BMO Integration User is a full administrative user on Phabricator | dklawren | 4 | -18/+39 | |
2018-02-28 | Bug 1440239 - PhabBugz - Assign a secure-revision | Piotr Zalewa | 4 | -12/+69 | |
2018-02-28 | Bug 1437384 - phabbugz_feed.pl in PhabBugz extension should be extended to ↵ | dklawren | 3 | -34/+181 | |
also query for new users in Phab | |||||
2018-02-27 | Bug 1441329 - Fix typos in the PhahBugz User.pm module | dklawren | 1 | -3/+3 | |
2018-02-26 | Bug 1437383: Create User.pm PhabBugz class for loading of a user object from ↵ | dklawren | 1 | -0/+194 | |
phabricator | |||||
2018-02-13 | Bug 1434438 - Refactor Revision.pm to use Moo for cleaner type checking | dklawren | 2 | -148/+156 | |
2018-02-13 | Bug 1434064 - Refactor Project.pm to use Moo for better type checking | dklawren | 3 | -159/+154 | |
2018-02-07 | Bug 1430259 - Update policy code in BMO PhabBugz extension to update custom ↵ | dklawren | 3 | -91/+295 | |
policy if a private bugs groups have changed. | |||||
2018-01-11 | Bug 1429600 - Update Revision.pm type checking to treat bug id as a simple ↵ | dklawren | 1 | -2/+5 | |
string or undefined | |||||
2018-01-09 | Bug 1429110: Update Revision.pm type checking to treat bug id as a simple string | dklawren | 1 | -7/+5 | |
2018-01-05 | Bug 1428156 - BMO will mark a revision as public if it sees a new one ↵ | dklawren | 1 | -2/+12 | |
created without a bug id associated with it | |||||
2018-01-03 | Bug 1426518 - Revisions can optionally not have a bug id so we need to make ↵ | dklawren | 4 | -25/+42 | |
it optional in the type constraints of Revision.pm | |||||
2018-01-03 | Bug 1427743 - legacy phabbugz API code errors when trying to set an inactive ↵ | dklawren | 1 | -2/+2 | |
review flag | |||||
2017-12-20 | Bug 1426424 - feed daemon complains when trying to set an inactive review flag | David Lawrence | 1 | -2/+2 | |
2017-12-19 | Bug 1422329 - The phabricator conduit API method feed.query_id return data ↵ | David Lawrence | 3 | -21/+13 | |
format has changed so the phabbugz_feed.pl daemon needs to be updated | |||||
2017-12-19 | Bug 1409957 - Create polling daemon to query Phabricator for recent ↵ | David Lawrence | 11 | -59/+1348 | |
transcations and update bug data according to revision changes | |||||
2017-12-16 | Bug 1403777 - Migrate urlbase from params to localconfig | Dylan William Hardison | 1 | -2/+2 | |
2017-12-05 | Revert "Bug 1409957 - Create polling daemon to query Phabricator for recent ↵ | Dylan William Hardison | 11 | -1348/+59 | |
transcations and update bug data according to revision changes" This reverts commit 7da8e374e0c96e10077690935e829b0c04fc82f4. | |||||
2017-12-05 | Revert "Bug 1422329 - The phabricator conduit API method feed.query_id ↵ | Dylan William Hardison | 3 | -13/+21 | |
return data format has changed so the phabbugz_feed.pl daemon needs to be updated" This reverts commit 357528ba856fd85aee5967e5d7d8f4426bc8d5b6. | |||||
2017-12-01 | Bug 1422329 - The phabricator conduit API method feed.query_id return data ↵ | David Lawrence | 3 | -21/+13 | |
format has changed so the phabbugz_feed.pl daemon needs to be updated | |||||
2017-11-29 | Bug 1409957 - Create polling daemon to query Phabricator for recent ↵ | dklawren | 11 | -59/+1348 | |
transcations and update bug data according to revision changes | |||||
2017-10-11 | Bug 1407617 - When making a phab revision public, the edit policy also needs ↵ | dklawren | 1 | -0/+4 | |
to be set to 'users' (#261) | |||||
2017-10-10 | Bug 1399593 - Add sync group comments during new revision creation as well ↵ | David Walsh | 2 | -22/+88 | |
as push connector | |||||
2017-09-28 | Bug 1403584: Clear r+ in bug when reviewer is removed from the revision by ↵ | dklawren | 1 | -4/+11 | |
the author via Change Reviewers (#251) | |||||
2017-09-14 | Bug 1399508 - Pass along transaction id to BMO API calls for linking back to ↵ | dklawren | 1 | -0/+2 | |
the exact change in the revision history (#237) | |||||
2017-09-12 | Bug 1396957 - Synchronize CC list on security bugs over to revisions as ↵ | dklawren | 1 | -1/+18 | |
subscribers (#230) * Bug 1396957 - Synchronize CC list on security bugs over to revisions as subscribers * - Updated to only make subscriber changes if no groups have changed. - This cuts down on the amount of custom policy changes in the revision activity. * Bug 1396957 - Synchronize CC list on security bugs over to revisions as subscribers * - Updated to only make subscriber changes if no groups have changed. - This cuts down on the amount of custom policy changes in the revision activity. * Bug 1396042 - Implement logging for the BMO push connector * Bug 1397747 - Move _get_all_group_names to Bugzilla::Config::Common::_get_all_group_names, and remove copy-pasted code | |||||
2017-09-11 | Bug 1397927 - Add comment to bug when a revision is accepted/denied with the ↵ | dklawren | 1 | -7/+31 | |
users real names (#233) Bug 1397927 - Add comment to bug when a revision is accepted/denied with the users real names * - Bug fixes * Wording changes. | |||||
2017-08-31 | Fix 1395685 - Update errant self reference in PhabBugz | David Walsh | 1 | -1/+1 | |
2017-08-31 | Bug 1395346 - Allow setting obsolete to false for phabricator attachments ↵ | dklawren | 1 | -2/+8 | |
when a user reclaims a revision (#227) | |||||
2017-08-30 | Revert "Bug 1395346 - Allow setting obsolete to false for phabricator ↵ | David Lawrence | 1 | -8/+2 | |
attachments when a user reclaims a revision" This reverts commit c48fede1691feb1056cd21422716f297af9e7bc3. | |||||
2017-08-30 | Bug 1395346 - Allow setting obsolete to false for phabricator attachments ↵ | David Lawrence | 1 | -2/+8 | |
when a user reclaims a revision | |||||
2017-08-30 | Bug 1393023: Approving a revision creates an r+ flag on the revision ↵ | dklawren | 2 | -9/+178 | |
attachment in the associated bug (#219) Approving a revision creates an r+ flag on the revision attachment in the associated bug * - Working version - Splits accepted_users on ':' instead of accepting a list (phab issue) | |||||
2017-08-23 | Bug 1390316 - Unused variable in PhabBugz script $phab_api_key | Mary Umoh | 1 | -5/+1 | |
2017-08-17 | Bug 1389372 - Add REST API endpoint to validate whether a BMO user ID can ↵ | David Walsh | 2 | -0/+52 | |
see a given BMO bug | |||||
2017-08-11 | Bug 1382225 - Missing code from PhabBugz extension such as http basic auth ↵ | dklawren | 3 | -28/+95 | |
support and other minor improvements | |||||
2017-08-01 | Bug 1385947 - Make revisions fully public (#189) | David Walsh | 1 | -2/+2 | |
2017-07-31 | Bug 1383108 - Sync security groups with Phabricator push connector | David Walsh | 2 | -1/+36 | |
r=dkl | |||||
2017-07-20 | Bug 1380727 - Update BMO Push Connector to Make Revisions Public when Bug ↵ | dklawren | 1 | -2/+2 | |
Made Public | |||||
2017-07-19 | Bug 1380727 - Update BMO Push Connector to Make Revisions Public when Bug ↵ | David Walsh | 2 | -7/+8 | |
Made Public | |||||
2017-07-10 | Bug 1379632 - Error is thrown by update_project_members.pl when Phabricator ↵ | David Lawrence | 1 | -1/+2 | |
returns a null value for a bmo user id search | |||||
2017-07-07 | Bug 1377933 - Remove trailing whitespace from all perl files | Dylan William Hardison | 1 | -1/+1 | |
2017-07-07 | Bug 1367604 - BMO extension to apply security policies to Phabricator ↵ | dklawren | 6 | -114/+426 | |
revisions as needed * - Updated based on dylans review - Fixed custom policy to instead allow projects and subscribers and then add BMO roles to the subscriber list - Some other bug fixes * fix lifetime of phabricator_url_re() Instead of passing the value (which depends on runtime configuration) pass in a reference. Also edit extensions/BMO/Extension.pm to allow %autodetect_attach_urls regex option to be a callback instead of just a plain regexp ref. * - Fixed regex in BMO extension to detect phabricator attachments - Use request_cache for useragent handle in Util.pm | |||||
2017-06-06 | Bug 1370378 - Add Phabricator Extension enabled flag to instantly toggle off ↵ | David Walsh | 4 | -0/+12 | |
all Phabricator syncing. r?dkl | |||||
2017-06-01 | Bug 1368066 - When using bmo auth delegation from Phabricator, the ↵ | David Lawrence | 3 | -0/+35 | |
confirmation should be skipped similar to mozreview |