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

DTSPB-4411: Back button with DTSPB-3650 spike #2318

Draft
wants to merge 28 commits into
base: master
Choose a base branch
from

Conversation

jamiealbertelli
Copy link
Contributor

Before creating a pull request make sure that:

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • update CSS version to force refresh if any changes are introduced

Please remove this line and everything above and fill the following sections:

JIRA link (if applicable)

Change description

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[ ] No

nameAsOnTheWill: get(formdata, 'applicant.nameAsOnTheWill'),
isApplying: true,
isApplicant: false
};
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pure Spike here, this is to cycle through x num of execs

@jamiealbertelli jamiealbertelli marked this pull request as draft December 4, 2024 11:30
CodicilsDamageDate: 'CodicilsDamageDate',
WillCodicils: 'WillCodicils'
},
ApplicantName: 'TaskList',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

multiExecPreviousStepList is the same here (L335) till the closing of this object.

}
};

const multiExecPreviousStepList = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

YAGNI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant