Skip to content

Commit

Permalink
Merge pull request #248 from opencrvs/auto-pr-release-v1.6.0-244-16719
Browse files Browse the repository at this point in the history
🍒 Merge changes from PR #244 to release-v1.6.0
  • Loading branch information
Zangetsu101 authored Sep 19, 2024
2 parents fb461e9 + 89abe56 commit 4216d30
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 12 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
- Remove `inputFieldWidth` from Number type form field
- You can now configure the home screen application’s name and icons in your country configuration package as manifest.json and app icon files are moved from core to country config (check `src/client-static` folder)
- Updated `allowedFileFormats` in signature fields to use MIME types (`image/png`, `image/jpg`, `image/jpeg`, `image/svg`) instead of simple file extensions. If you are already using the `allowedFileFormats` field in your implementation, please ensure to update the format accordingly.
- Remove unnecessary UI dividers that add in various sections of the declaration forms(e.g the Death, Birth and Marriage forms) [#244](https://github.com/opencrvs/opencrvs-countryconfig/pull/244)

### Bug fixes

Expand Down
11 changes: 7 additions & 4 deletions src/form/addresses/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,8 @@ export const defaultAddressConfiguration: IAddressConfiguration[] = [
},
{
// MOTHER ADDRESS FIELDS
precedingFieldId: 'birth.mother.mother-view-group.mother-nid-seperator',
precedingFieldId:
'birth.mother.mother-view-group.motherBirthRegistrationNumber',
configurations: [
{
config: AddressSubsections.PRIMARY_ADDRESS_SUBSECTION,
Expand All @@ -117,7 +118,8 @@ export const defaultAddressConfiguration: IAddressConfiguration[] = [
},
{
// FATHER ADDRESS FIELDS
precedingFieldId: 'birth.father.father-view-group.father-nid-seperator',
precedingFieldId:
'birth.father.father-view-group.fatherBirthRegistrationNumber',
configurations: [
{
config: AddressSubsections.PRIMARY_ADDRESS_SUBSECTION,
Expand Down Expand Up @@ -266,7 +268,8 @@ export const defaultAddressConfiguration: IAddressConfiguration[] = [
},*/
{
// SPOUSE ADDRESS FIELDS
precedingFieldId: 'death.spouse.spouse-view-group.spouse-nid-seperator',
precedingFieldId:
'death.spouse.spouse-view-group.spouseBirthRegistrationNumber',
configurations: [
{
config: AddressSubsections.PRIMARY_ADDRESS_SUBSECTION,
Expand Down Expand Up @@ -301,7 +304,7 @@ export const defaultAddressConfiguration: IAddressConfiguration[] = [
{
// PLACE OF MARRIAGE ADDRESS FIELDS
precedingFieldId:
'marriage.marriageEvent.marriage-event-details.place-of-marriage-seperator',
'marriage.marriageEvent.marriage-event-details.placeOfMarriageTitle',
configurations: [{ config: EventLocationAddressCases.PLACE_OF_MARRIAGE }]
},
{
Expand Down
4 changes: 0 additions & 4 deletions src/form/birth/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -353,8 +353,6 @@ export const birthForm: ISerializedForm = {
hideIfNidIntegrationEnabled.concat(detailsExist),
true
),
// preceding field of address fields
divider('mother-nid-seperator', detailsExist),
// ADDRESS FIELDS WILL RENDER HERE
divider('mother-address-seperator', detailsExist),
getMaritalStatus(certificateHandlebars.motherMaritalStatus, [
Expand Down Expand Up @@ -442,8 +440,6 @@ export const birthForm: ISerializedForm = {
hideIfNidIntegrationEnabled.concat(detailsExist),
true
),
// preceding field of address fields
divider('father-nid-seperator', detailsExist),
// ADDRESS FIELDS WILL RENDER HERE
divider('father-address-seperator', detailsExist),
getMaritalStatus(certificateHandlebars.fatherMaritalStatus, [
Expand Down
2 changes: 0 additions & 2 deletions src/form/death/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -364,8 +364,6 @@ export const deathForm = {
),
getIDType('death', 'spouse', detailsExist, true),
...getIDNumberFields('spouse', detailsExist, true),
// preceding field of address fields
divider('spouse-nid-seperator', detailsExist),
// ADDRESS FIELDS WILL RENDER HERE
divider('spouse-address-separator')
],
Expand Down
3 changes: 1 addition & 2 deletions src/form/marriage/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -264,8 +264,7 @@ export const marriageForm: ISerializedForm = {
fields: [
getMarriageDate, // Required field
getTypeOfMarriage,
placeOfMarriageSubsection,
divider('place-of-marriage-seperator')
placeOfMarriageSubsection
// PLACE OF MARRIAGE FIELDS WILL RENDER HERE
]
}
Expand Down

0 comments on commit 4216d30

Please sign in to comment.