Closed Bug 1401810 Opened 7 years ago Closed 5 years ago

[PulseGuardian] Add migration to drop owner_id from pulse_users

Categories

(Webtools :: Pulse, enhancement)

Product:
Webtools
Component:
Pulse
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: mcote, Unassigned)

Details

In bug 1068447 we added a new table, pulse_user_owners, to record owners of pulse users, so that there could be a many-to-many relationship.  However the owner_id column is still in the pulse_users table, unused, which caused me some confusion when I forgot about pulse_user_owners and tried to update owner_id.  We should add a migration to drop that column (and execute it on production).

Note that sqlite doesn't support dropping columns, but sqlite should just be used for local development so migrations aren't particularly important there.  We could add a more complex migration for sqlite that creates a new table without the column, copies the data over, drop the original table, and renames the new table, but I doubt it's worth it.
Summary: Add migration to drop owner_id from pulse_users → [PulseGuardian] Add migration to drop owner_id from pulse_users
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.