Closed Bug 1089936 Opened 10 years ago Closed 10 years ago

[rbbz] Remove login-group restriction

Categories

(MozReview Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mcote, Unassigned)

Details

Attachments

(1 file)

We're just about ready to open MozReview to the public.  Any BMO user will be able to use the system.
/r/332 - Bug 1089936 - Remove login-group restriction.

Pull down this commit:

hg pull review -r 1f51912563b68075fefdf8ba3c8869e378cf1607
/r/332 - Bug 1089936 - Remove login-group restriction.

Pull down this commit:

hg pull review -r 1f51912563b68075fefdf8ba3c8869e378cf1607
Attachment #8512384 - Flags: review?(gps)
/r/332 - Bug 1089936 - Remove login-group restriction.

Pull down this commit:

hg pull review -r 1f51912563b68075fefdf8ba3c8869e378cf1607
Sorry for the spam; this might be another bug, if you publish an already-published review request but with no actual changes.
Attachment #8512384 - Flags: review?(gps) → review+
https://reviewboard-dev.allizom.org/r/331/#review219

LGTM.

There are still a few references to this group in the test code that could be deleted as well.
https://hg.mozilla.org/hgcustom/version-control-tools/file/89f12dc59e95/testing/vcttesting/mozreview.py#l87
https://hg.mozilla.org/hgcustom/version-control-tools/file/89f12dc59e95/hgext/reviewboard/tests/helpers.sh#l65

r+ carried forward to delete those (but please run tests before landing just in case).
Removed those two lines and ran tests; they still pass.

http://hg.mozilla.org/hgcustom/version-control-tools/rev/b93beffe770f
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Version: Trunk → unspecified
Product: Developer Services → MozReview
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: