Skip to content

[IN-137] chore: add deleted files updated at index and users email index#1036

Merged
apsantiso merged 3 commits into
masterfrom
chore/add-deleted-files-updated-at-index
Apr 17, 2026
Merged

[IN-137] chore: add deleted files updated at index and users email index#1036
apsantiso merged 3 commits into
masterfrom
chore/add-deleted-files-updated-at-index

Conversation

@apsantiso
Copy link
Copy Markdown
Collaborator

What

Index to keep performant the hard delete oldest deleted files.

@apsantiso apsantiso requested a review from sg-gs April 15, 2026 13:34
@apsantiso apsantiso requested a review from jzunigax2 as a code owner April 15, 2026 13:34
@apsantiso apsantiso added the migration-required A migration is included in the PR label Apr 15, 2026
sg-gs
sg-gs previously approved these changes Apr 15, 2026
Copy link
Copy Markdown
Member

@sg-gs sg-gs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Running migration @apsantiso

@sg-gs
Copy link
Copy Markdown
Member

sg-gs commented Apr 16, 2026

Migration run @apsantiso

@apsantiso
Copy link
Copy Markdown
Collaborator Author

@sg-gs I didn't notice the PR for the users email was using 20260415132931-create-deleted-files-updated-at-index.js as name.

You already ran 20260415132931-create-deleted-files-updated-at-index.js with the correct index from this PR so I just added the index for users email (which is already created but we didn't add the migration) here to fix the migration.

@apsantiso apsantiso requested a review from sg-gs April 16, 2026 15:49
@apsantiso apsantiso changed the title [IN-137] chore: add deleted files updated at index [IN-137] chore: add deleted files updated at index and users email index Apr 16, 2026
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Member

@sg-gs sg-gs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Running the last migration @apsantiso

@sg-gs
Copy link
Copy Markdown
Member

sg-gs commented Apr 17, 2026

Done @apsantiso

Sequelize CLI [Node: 22.16.0, CLI: 6.6.3, ORM: 6.37.7]

Loaded configuration file "src/config/sequelize.js".
Using environment "production".
== 20260416154502-create-user-email-index: migrating =======
== 20260416154502-create-user-email-index: migrated (0.091s)

Done in 1.29s.

@apsantiso apsantiso merged commit ddcd3e6 into master Apr 17, 2026
9 checks passed
@apsantiso apsantiso deleted the chore/add-deleted-files-updated-at-index branch April 17, 2026 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

migration-required A migration is included in the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants