Skip to content

Show unit ID on details page and in form (i18n)#300

Merged
stefan-burke merged 3 commits intomainfrom
terragon/add-unitid-to-unit-details-256dqt
Oct 13, 2025
Merged

Show unit ID on details page and in form (i18n)#300
stefan-burke merged 3 commits intomainfrom
terragon/add-unitid-to-unit-details-256dqt

Conversation

@stefan-burke
Copy link
Copy Markdown
Member

Summary

  • Adds unit ID display on the unit details page using a localized label
  • Renders the unit ID in the unit form for persisted records

Changes

UI

  • In app/views/shared/_unit_details.html.erb: inserted a new list item showing the unit ID with label t("units.fields.id").
  • In app/views/units/_form.html.erb: within the data fieldset, render the ID field via the existing display_field partial for :id.

i18n

  • Uses i18n key units.fields.id for the label; ensure a translation exists for the target locale (e.g., en).

Tests

  • Verify the details page shows the ID label and value
  • Verify the edit form displays the ID field for persisted units

Notes

  • No functional changes beyond exposing the unit ID in details and edit forms.

🌿 Generated by Terry


ℹ️ Tag @terragon-labs to ask questions and address PR feedback

📎 Task: https://www.terragonlabs.com/task/a2645f27-5d95-4124-9c23-9957af45b6bf

stefan-burke and others added 3 commits October 13, 2025 00:19
Added a list item to the _unit_details.html.erb partial to show the unit's ID field to provide more detailed information about each unit.

Co-authored-by: terragon-labs[bot] <terragon-labs[bot]@users.noreply.github.com>
Added rendering of the unit's ID within the data fieldset in the units form partial. This provides a read-only display of the unit's identifier when the unit is persisted.

Co-authored-by: terragon-labs[bot] <terragon-labs[bot]@users.noreply.github.com>
@stefan-burke stefan-burke merged commit ec66835 into main Oct 13, 2025
14 of 15 checks passed
@stefan-burke stefan-burke deleted the terragon/add-unitid-to-unit-details-256dqt branch October 13, 2025 00:26
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.

1 participant