Skip to content

[WIP] Update project_users to use title from position data#735

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/convert-position-to-title-data
Draft

[WIP] Update project_users to use title from position data#735
Copilot wants to merge 2 commits intomainfrom
copilot/convert-position-to-title-data

Conversation

Copy link
Contributor

Copilot AI commented Jan 20, 2026

  • Rebase to current main (organization_people replaced project_users)
  • Create migration to copy position enum values to title (where title is nil)
  • Update model to remove position enum
  • Update views that reference position to use title
  • Update models that query by position value
  • Update controllers that permit position parameter
  • Update factories to use title instead of position
  • Update tests to remove position enum expectations
  • Run tests to verify changes
  • Run linting to ensure code quality
Original prompt

This section details on the original issue you should resolve

<issue_title>Convert position data to title data</issue_title>
<issue_description>Stakeholders are no longer using Position data for project_users -- they're allowing free text for a title within external filemakerpro db

  • Update project_users.title to project_users.position value (using enum) wherever title is nil
  • Drop position column
  • Remove enum from project_user.rb</issue_description>

<agent_instructions>write code to complete this story.

some additional details to the description: production has job title data in project_users.position.

write code to migrate the existing data into the project_users.title field, drop the column, and remove the enum since it's now a free-text field.</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>
@maebeale
Copy link
Collaborator

@copilot see if you can start with current main and apply these commits. project_users was renamed to organization_people.

Copy link
Contributor Author

Copilot AI commented Feb 15, 2026

@copilot see if you can start with current main and apply these commits. project_users was renamed to organization_people.

I've rebased the changes to current main and applied them to organization_people instead of project_users. The changes are ready in commit 6cd3870:

  • Created migration to convert position enum values to title strings
  • Removed position enum from OrganizationPerson model
  • Updated Organization and Person models to query by title instead of position
  • Updated all views to use title instead of position
  • Updated controllers to remove position from permitted parameters
  • Updated factories and tests

The branch is based on current main (80489be) which has the organization_people model. However, the remote branch still has the old project_users changes. A force push will be needed to update the PR with these new changes.

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.

Convert position data to title data

2 participants