Skip to content

[FIX] Improve avatar migration#7352

Merged
rodrigok merged 4 commits into
release-candidatefrom
improve-avatar-migration
Jun 28, 2017
Merged

[FIX] Improve avatar migration#7352
rodrigok merged 4 commits into
release-candidatefrom
improve-avatar-migration

Conversation

@sampaiodiego

Copy link
Copy Markdown
Member

@RocketChat/core

Fix overall issues and run asynchronously which increases performance considerably.

@sampaiodiego sampaiodiego added this to the 0.57.0 milestone Jun 27, 2017
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-7352 June 27, 2017 21:46 Inactive
@@ -1,5 +1,5 @@
if (RocketChat.Migrations.getVersion() !== 0) {
RocketChat.Migrations.migrateTo('latest');
RocketChat.Migrations.migrateTo(process.env.MIGRATION_VERSION || 'latest');

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

wait... does this mean we can trigger this to re-run a migration?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

yes, but it didn't work as I expected.. it needs more work before we can spread the word. 😬

for example, to run migration 97 again (avatars migration) I had to set MIGRATION_VERSION=98,rerun which does not make much sense 🤷‍♂️

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤔 that's kinda weird...

@rodrigok rodrigok merged commit 7ccaa03 into release-candidate Jun 28, 2017
@rodrigok rodrigok deleted the improve-avatar-migration branch June 28, 2017 14:31
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.

4 participants