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

Add village and other fields based on kobo form changes #171

Merged
merged 15 commits into from
Sep 23, 2024

Conversation

wadhwamatic
Copy link
Member

@wadhwamatic wadhwamatic commented Sep 19, 2024

Add new village field to the three forms

Update Incident report to add a new shipwreck incident type and factory counts
Update Flood report to add 3 new fields

Update Readme to guide users through column additions

@wadhwamatic wadhwamatic changed the base branch from main to new-ui-2024 September 19, 2024 01:59
@@ -86,6 +86,8 @@ export class IncidentDto {
@IsOptional() @IsNumberString() readonly 'group_pk03i04/group_ub2hp14/CraftDam'?: string;
@IsOptional() @IsNumberString() readonly 'group_gh6ag70/group_na4yi04/CropAff'?: string;
@IsOptional() @IsNumberString() readonly 'group_gh6ag70/group_na4yi04/CropDam'?: string;
@IsOptional() @IsNumberString() readonly 'group_gh6ag70/group_az7bv13/integer_ws0la67'?: string;
Copy link
Collaborator

Choose a reason for hiding this comment

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

@wadhwamatic I think they should update these names before they get integrated in other apps to stay consistent with the rest

Copy link
Member Author

Choose a reason for hiding this comment

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

agreed. I'll have them update it in kobo

@@ -211,8 +211,13 @@
"NumWareHDam": "Damaged",
"NumCraftAff": "Affected",
"NumCraftDam": "Damaged",
"integer_ws0la67": "Affected",
Copy link
Collaborator

@ericboucher ericboucher Sep 20, 2024

Choose a reason for hiding this comment

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

These do not seem to appear in Flood, should they be under Incident instead?

@@ -30,6 +30,7 @@ export const IncidentMapping: MappingTypes = {
drowning: '10',
collapse: '11',
weapon: '12',
shipwreck: '13',
Copy link
Collaborator

Choose a reason for hiding this comment

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

I also saw a value 14 in the files you shared, should we add it too?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks like it's also shipwreck (based on google trad) so I'm a bit confused

@@ -132,6 +134,8 @@ export const incidentSpecificKeys = {
[IncidentSpecific.CraftDam]: 'group_pk03i04/group_ub2hp14/CraftDam',
[IncidentSpecific.CropAff]: 'group_gh6ag70/group_na4yi04/CropAff',
[IncidentSpecific.CropDam]: 'group_gh6ag70/group_na4yi04/CropDam',
[IncidentSpecific.integer_ws0la67]: 'group_gh6ag70/group_az7bv13/CropAff',
Copy link
Collaborator

@ericboucher ericboucher Sep 20, 2024

Choose a reason for hiding this comment

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

The values on the right should be the corresponding keys from the kobo forms

Copy link

github-actions bot commented Sep 20, 2024

Build succeeded and deployed at https://wfp-dmp-171.surge.sh
(hash 9772224 deployed at 2024-09-23T09:01:44)

@ericboucher ericboucher changed the title Add new fields based on kobo form changes Add village and other fields based on kobo form changes Sep 23, 2024
@ericboucher ericboucher merged commit 1b9df6c into new-ui-2024 Sep 23, 2024
3 checks passed
@ericboucher ericboucher deleted the form-updates branch September 23, 2024 19:01
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