Unify push to jira logic for finding/groups review/open/close

[Maffooch]

Sometimes when working with the jira integration, there are instances of a finding being pushed to jira when modifying the status of a finding via the edit page, but not when using the open/close/review buttons. This is confusing and creates disparity.

Here is what this PR does

• Update single/grouped findings in Jira when a single finding is closed/open
• Update single/grouped findings in Jira when a single finding has a requested review, cleared review, or actually reviewed
• Correct statuses of findings under review to be Active instead of Inactive. This makes more sense as the decision for whether the finding is valid/mitigated is still up for debate and should not be marked as inactive until confirmed via review
• Correct statuses of findings that have been reviewed when "not fixed" was selected. Previously, the finding would not be updated and would stay in an inactive state. This is crazy. The behavior now is the finding will be active and verified since it is confirmed the finding was not actually fixed and should this fact has been verified.
• Correct statuses of findings that have been reviewed when "Closed" was selected. Previously, the finding would not be marked as verified, but that is the whole point of the review. So now the finding will be marked as inactive, mitigated, and verified.
• Express Jira instance creation would not save the "automatically sync findings" flag. It does now
  [sc-1040]

[Maffooch]

Do not merge until @37b has a chance to review

[37b]

Makes sense to me.