Skip to content

(chore) Normalize ampathforms filenames#67

Merged
denniskigen merged 1 commit into
openmrs:mainfrom
denniskigen:fix/normalize-ampathforms-filenames
Apr 24, 2026
Merged

(chore) Normalize ampathforms filenames#67
denniskigen merged 1 commit into
openmrs:mainfrom
denniskigen:fix/normalize-ampathforms-filenames

Conversation

@denniskigen
Copy link
Copy Markdown
Member

@denniskigen denniskigen commented Apr 24, 2026

Summary

Rename form and translation files under configuration/backend_configuration/ampathforms/ and configuration/backend_configuration/ampathformstranslations/ to a consistent shape:

  • snake_case throughout (no spaces, no PascalCase, no hyphens within the base name)
  • -core_demo package suffix applied uniformly (was previously on 5 of 8 forms)
  • Fix typo: stuctured_soap_notestructured_soap_note

Renames

ampathforms/

Old New
Mental Health Assessment Form.json mental_health_assessment_form-core_demo.json
TestResultsEntryFormv2-core_demo.json test_results_entry_form_v2-core_demo.json
ipd_admission_request.json ipd_admission_request-core_demo.json
stuctured_soap_note.json structured_soap_note-core_demo.json
test-form-core_demo.json test_form-core_demo.json

ampathformstranslations/

Old New
Test Form 1_translations_fr-core_demo.json test_form_1_translations_fr-core_demo.json
soap_note_translations_en.json soap_note_translations_en-core_demo.json
soap_note_translations_fr.json soap_note_translations_fr-core_demo.json

Safety

No content changes. Internal name and uuid fields inside each JSON are untouched, so:

Note on the translation filename test_form_1_translations_fr-core_demo.json: its internal form field is "Test Form 1", which matches the internal name of test_form-core_demo.json. The filenames differ only in the trailing _1, which is cosmetic — binding is by internal name.

Out of scope

Rename form and translation files under
configuration/backend_configuration/ampathforms/ and
ampathformstranslations/ to consistent snake_case, apply the
-core_demo package suffix uniformly, and fix the "stuctured" →
"structured" typo.

Internal form names and UUIDs are not changed, so runtime form
lookup and translation binding (which use the form's internal
`name` field) are unaffected.
@denniskigen denniskigen merged commit c5d4682 into openmrs:main Apr 24, 2026
1 check passed
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