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.

Sign up for GitHub

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

Jump to bottom

inline error #1728

Open
wants to merge 60 commits into
base: master
Choose a base branch
from
Open

inline error #1728

wants to merge 60 commits into from

Conversation

sealDebolina
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)
  • the enable_keep_helm label has been added, if the helm release should be persisted after a successful build

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

hcarslaw and others added 30 commits November 29, 2024 12:18
@hcarslaw
Clean C100 data
7ac8d85
@vivek-sekhar
Merge branch 'master' into c100-data-cleanup
19e2a9e
@vivek-sekhar
feat(PRL-6545): clean up C100 data
64aaf82
@vivek-sekhar
feat(PRL-6545): clean up C100 data
66302e3
@vivek-sekhar
Merge branch 'master' into c100-data-cleanup
c600e78
@hcarslaw
Fix clearing c1a passport other details when no children have passports
c96cb43
@sealDebolina
inline error
375ea60
@hcarslaw
Fix merge conflicts
6f0a451
@hcarslaw
Unskip tests
bc9c896
@hcarslaw
Add redirects to CYA for C100 application sections
e4bdf2e
@hcarslaw
Merge branch 'master' into c100-value-add
c00b205
@hcarslaw
Fix merge conflicts
83a3de0
@hcarslaw
Merge branch 'master' into c100-value-add
7eef3d3
@vivek-sekhar
Add new framework for mandatory fields for c100 app
ccdc59b
@hcarslaw
Merge branch 'master' into c100-value-add
37342be
@hcarslaw
Merge branch 'c100-value-add' into inline-error
cbb17e1
@hcarslaw
Merge branch 'c100-value-add' into c100-section-redirect
2e2f77b
@hcarslaw
Add inline and top level errors
175a93c
@sealDebolina
Merge branch 'master' into c100-value-add
74eb254
@sealDebolina
Merge branch 'c100-value-add' into inline-error
4fe9208
@sealDebolina
Merge branch 'master' into c100-value-add
25393dc
@sealDebolina
Merge branch 'c100-value-add' into inline-error
da96ede
@hcarslaw
Merge branch 'c100-value-add' into c100-section-redirect
3c9b767
@hcarslaw
fix local development
60ec042
@hcarslaw
Add configs for people, miam and consent order sections
b3d5a8e
@hcarslaw
Merge branch 'c100-value-add' into c100-section-redirect
4578cfb
@sealDebolina
Merge branch 'c100-value-add' into inline-error
d0ec698
@hcarslaw
Fix review comments
56c76b9
@sealDebolina
top error+test case
e0ada0a
@hcarslaw
Merge pull request #1741 from hmcts/c100-section-redirect
15f8c4e 
Add redirects to CYA for C100 application sections
@hcarslaw hcarslaw changed the base branch from c100-value-add to master January 31, 2025 13:15
@sealDebolina sealDebolina changed the base branch from master to c100-value-add February 3, 2025 09:44
@hcarslaw hcarslaw changed the base branch from c100-value-add to master February 7, 2025 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enable_keep_helm ns:private-law prd:prl rel:prl-citizen-frontend-pr-1728
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sealDebolina @hcarslaw @vivek-sekhar