Skip to content

Show FileMaker code instead of internal ID on org form#1344

Open
maebeale wants to merge 1 commit intomainfrom
maebeale/org-filemaker-code
Open

Show FileMaker code instead of internal ID on org form#1344
maebeale wants to merge 1 commit intomainfrom
maebeale/org-filemaker-code

Conversation

@maebeale
Copy link
Collaborator

@maebeale maebeale commented Mar 5, 2026

What is the goal of this PR and why is this important?

  • Replace the internal ID field with the FileMaker code field on the organization form so admins can view/edit the FileMaker Project ID directly
  • Add a data migration rake task to move existing internal_id values into filemaker_code

How did you approach the change?

  • Swapped internal_id for filemaker_code in the org form (admin-only field) with label "FileMaker Project ID"
  • Updated strong params to permit filemaker_code instead of internal_id
  • Added rake organizations:migrate_internal_id_to_filemaker_code which:
    • Copies stripped internal_id into filemaker_code when filemaker_code is blank and internal_id is present
    • Adds a comment to the organization when both fields have values that don't match after stripping
    • Skips records where values already match

UI Testing Checklist

  • Verify "FileMaker Project ID" field appears on org edit form for admin users
  • Verify field is hidden for non-admin users
  • Verify saving a value to the field persists correctly

Anything else to add?

  • Run rake organizations:migrate_internal_id_to_filemaker_code in production after deploy to migrate existing data

🤖 Generated with Claude Code

- Replace internal_id field with filemaker_code on organization form
- Update strong params to permit filemaker_code instead of internal_id
- Add rake task to migrate existing internal_id values to filemaker_code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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