Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Check your claim" page fixes #2375

Merged
merged 4 commits into from
May 24, 2018
Merged

"Check your claim" page fixes #2375

merged 4 commits into from
May 24, 2018

Conversation

lostie
Copy link
Contributor

@lostie lostie commented May 22, 2018

What

  • Ensure that, for litigator interim claims, disbursements are added to the claim totals if changed before validations are performed
  • Display disbursement section for litigator interim claims
  • Display offence category for LGFS claims in the CYC page

Relates to: #2340

Ticket

Check your claim page fixes

Some of these

NOTE: This PR does not include all the fixes for the ticket, as otherwise it would turn up to be a massive one again, so I'm trying to deliver small pieces at a time.

Sergio Marques added 2 commits May 22, 2018 13:26
For interim claims, if disbursements only are supplied then update
totals with disbursements totals before validations are performed so the
validations work as expected and the user does not get blocked on the
page with a total value claimed error when they have submitted the
correct information.
LGFS interim claim do not have a warrant fee association but instead can
have an interim fee which is of type warrant.
@ghost ghost assigned lostie May 22, 2018
@ghost ghost added the in progress label May 22, 2018
@lostie lostie added the WIP label May 22, 2018
Sergio Marques added 2 commits May 22, 2018 16:46
disbursement section is actual part of the misc fees steps, so if it
needs to be edited, the user will see the misc fees step.
@lostie lostie force-pushed the check-your-claim-page-fixes branch 2 times, most recently from 55bae7b to 2224907 Compare May 24, 2018 09:58
@lostie lostie requested review from jsugarman and colinbruce May 24, 2018 10:02
Copy link
Contributor

@colinbruce colinbruce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM

@lostie lostie merged commit e6611ef into master May 24, 2018
@ghost ghost removed the ready for review label May 24, 2018
@lostie lostie deleted the check-your-claim-page-fixes branch May 24, 2018 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants