Page MenuHomePhabricator

Make "JIRA Issues" field work better with noncredentialed accounts
ClosedPublic

Authored by epriestley on Apr 2 2014, 6:34 PM.
Tags
None
Referenced Files
F15472683: D8676.id20570.diff
Sat, Apr 5, 3:27 PM
F15472432: D8676.diff
Sat, Apr 5, 1:33 PM
F15464728: D8676.id20570.diff
Wed, Apr 2, 12:38 PM
F15441865: D8676.id20570.diff
Wed, Mar 26, 8:30 PM
F15425409: D8676.id.diff
Sun, Mar 23, 4:36 AM
F15423293: D8676.id20570.diff
Sat, Mar 22, 2:32 PM
F15420550: D8676.diff
Fri, Mar 21, 2:54 PM
F15417342: D8676.id20570.diff
Thu, Mar 20, 4:42 PM

Details

Summary

Currently, users get an error when making any changes to this field if they don't have a linked JIRA account.

Instead:

  • We should only raise an error if they're trying to add issues, and only on the new issues. It's always fine to remove issues, and existing issues the author can't see are also fine.
  • When we can't add things because there's no account (vs because there's a permissions error or they don't exist), raise a more tailored exception.
Test Plan
  • As JIRA and non-JIRA users, made various edits to this field.
  • Got appropriate exceptions, including better tailoring.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Make "JIRA Issues" field work better with noncredentialed accounts.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Apr 2 2014, 6:40 PM
epriestley updated this revision to Diff 20574.

Closed by commit rP81fa847bc52b (authored by @epriestley).