Skip to content

Commit

Permalink
[PP-17] Show detail error message if JQL is in wrong format
Browse files Browse the repository at this point in the history
  • Loading branch information
Yiming Bao committed Apr 11, 2016
1 parent 7320351 commit adf886a
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
3 changes: 2 additions & 1 deletion PlanningPoker/Changelog.txt
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ Released: 2016-03-23
the aggregated value, the font color should be DarkKhaki.
3) If none of the sub tasks has specified story points, then skip.
* Enable to save encrypted password
* Show detail error message if http response code is not OK

== Bug Fixes ==
* Fixed issue PP-16: When assignee is empty or there is no sub task, an NullReferenceException will be thrown
Expand All @@ -34,7 +35,7 @@ Released: 2016-03-08
* Show sub task under parent cards
* Show story point in story list
* Add one algorithm to calculate story point by role group.
Note only Dev and QA are valid in the algorithm. In the screenshot below, the left number (8) is Dev, the right number (8) is QA.
Note only Dev and QA are eligible in the algorithm. In the screenshot below, the left number (8) is Dev, the right number (8) is QA.
You should specify your role before joining the game.
To enable this feature, you need to uncomment out the following item in PlanningPoker.exe.config
<add key="StoryPointAlgorithm" value="RoleGroup"/>
Expand Down
9 changes: 8 additions & 1 deletion PlanningPoker/Utility/RestUtil.cs
Original file line number Diff line number Diff line change
Expand Up @@ -54,9 +54,16 @@ public class RestUtil
throw new UnauthorizedAccessException("JIRA access forbidden");
}

if (response.StatusCode >= HttpStatusCode.BadRequest)
{
log.Error(response.StatusCode);
throw new WebException(response.Content);
}

if (response.ErrorException != null)
{
log.Error(string.Format("error get,url={0},status={1},exception={2}", url, response.StatusCode, response.ErrorException));
log.Error(string.Format("error get,url={0},status={1},exception={2}",
url, response.StatusCode, response.ErrorException));
throw response.ErrorException;
}

Expand Down

0 comments on commit adf886a

Please sign in to comment.