Merge changes from topic "remove-draft-workflow"

* changes:
  Remove "My Drafts" menu items from all user preferences
  ChangeStatusPredicate: Silently ignore invalid status
  ChangeStatusPredicate: Make protected fields private
  PolyGerrit: Remove support for drafts
  Discontinue draft workflow
This commit is contained in:
Dave Borowitz
2017-09-30 14:29:49 +00:00
committed by Gerrit Code Review
145 changed files with 598 additions and 3059 deletions

View File

@@ -54,11 +54,9 @@ limitations under the License.
test('fix my menu item', () => {
assert.deepEqual([
{url: '#/q/owner:self+is:draft'},
{url: 'https://awesometown.com/#hashyhash'},
{url: 'url', target: '_blank'},
].map(element._fixMyMenuItem), [
{url: '/q/owner:self+is:draft', external: true},
{url: 'https://awesometown.com/#hashyhash', external: true},
{url: 'url', external: true},
]);
@@ -66,12 +64,10 @@ limitations under the License.
test('filter unsupported urls', () => {
assert.deepEqual([
{url: '/q/owner:self+is:draft'},
{url: '/c/331788/'},
{url: '/groups/self'},
{url: 'https://awesometown.com/#hashyhash'},
].filter(element._isSupportedLink), [
{url: '/q/owner:self+is:draft'},
{url: '/c/331788/'},
{url: 'https://awesometown.com/#hashyhash'},
]);

View File

@@ -45,7 +45,6 @@
'is:',
'is:abandoned',
'is:closed',
'is:draft',
'is:ignored',
'is:mergeable',
'is:merged',
@@ -75,7 +74,6 @@
'status:',
'status:abandoned',
'status:closed',
'status:draft',
'status:merged',
'status:open',
'status:pending',