Skip to content

[PB-6385] Add migration to remove constraint#65

Merged
TamaraFinogina merged 1 commit into
masterfrom
remove_constraint_from_db
May 20, 2026
Merged

[PB-6385] Add migration to remove constraint#65
TamaraFinogina merged 1 commit into
masterfrom
remove_constraint_from_db

Conversation

@TamaraFinogina
Copy link
Copy Markdown
Contributor

@TamaraFinogina TamaraFinogina commented May 20, 2026

Migration to remove ['user_id', 'room_id'] uniqueness constraint from the database.

To fix user kick out the idea is to:

  1. Each joinCall will create a record in the database with (id, user_id, room_id, jitsi_id) and return id to the user ← this is why the constraint should be removed
  2. User stores id in sessionStorage
  3. When user is kicked out - Jitsi webhook sends jitsi_id for deletion
  4. When user is kicked out - meet-web sends leaveCall with id

3 and 4 will result in deletion of the same line (whichever arrives first) and mostly done for redundancy.

@TamaraFinogina TamaraFinogina self-assigned this May 20, 2026
@sonarqubecloud
Copy link
Copy Markdown

@TamaraFinogina TamaraFinogina requested a review from sg-gs May 20, 2026 15:23
@TamaraFinogina TamaraFinogina marked this pull request as ready for review May 20, 2026 15:23
@TamaraFinogina TamaraFinogina requested a review from jzunigax2 as a code owner May 20, 2026 15:23
@sg-gs
Copy link
Copy Markdown
Member

sg-gs commented May 20, 2026

Migration run @TamaraFinogina

Sequelize CLI [Node: 22.16.0, CLI: 6.6.2, ORM: 6.37.6]

Loaded configuration file "src/config/sequelize.js".
Using environment "production".
== 20260520142639-remove-unique-constraint-to-room-users: migrating =======
== 20260520142639-remove-unique-constraint-to-room-users: migrated (0.024s)

Done in 0.84s.

@TamaraFinogina TamaraFinogina merged commit 772f610 into master May 20, 2026
7 checks passed
@TamaraFinogina TamaraFinogina deleted the remove_constraint_from_db branch May 20, 2026 15:30
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