Conversation
There was a problem hiding this comment.
@sergii-tkachenko usually flags these to use ===
|
Just confirming, so once a Reportback Item is promoted, the whole Reportback is considered promoted? If an editor makes a mistake, will this remove the promoted flag upon changing the Reportback Item status? |
There was a problem hiding this comment.
Should we rename this, since it sets Promoted as well as Flagged? Maybe something like setFlags ?
There was a problem hiding this comment.
yeah, I thought about that, but it's not really flags I couldn't come up with a good name for it...
There was a problem hiding this comment.
Yeah.. hmm. setProperties... setExtras... setFlaggedPromoted ... ?
There was a problem hiding this comment.
loving setFlaggedPromoted
|
@aaronschachter yeah, it works the same way flagged does, the |
|
👍 |
|
@angaither - Random thought around this came to mind. What if a reportback has two items we want to promote, but for different reasons? Once a reportback item is flagged, the reportback is ignored entirely, so there would never be the opportunity to update the flagged reason upon adding additional litems. With this its different... wouldnt' the last promoted item always overwrite the reason? |
2 participants
Add in promoted reasons checkboxes
Fixes #4496