sprintly / sprintly-kanban

A Kanban Board for Sprintly
https://kanban.sprint.ly
ISC License
494 stars 82 forks source link

fix issue with downgrading statuses for items with a close reason #9

Closed wookiehangover closed 9 years ago

wookiehangover commented 9 years ago

What does it do?

Fixes an API error received when sending a close_reason for certain status changes. For instance, if you try to move an item from Completed to Accepted, it will error because the previous close_reason value is sent along to the API.

Where should the reviewer start?

small change in app/stores/product-store.js and some tests

How should this be manually tested?

Moving items from the Completed column should persist to the API.

Background context?

I thought this was an API bug but it turned out to be user error.

GIF?

tumblr_ml68tlx4xm1r9uuuso1_r1_500

florapdx commented 9 years ago

:+1: Confirmed fix.