[server] Introduce migration that adds baseRoutePath to AppURLFacts
ClosedPublic
Actions

Authored by ashoat on Mar 29 2022, 3:23 PM.

Details

Reviewers

varun
atul
tomek

Commits

rCOMM0122555af744: [server] Introduce migration that adds baseRoutePath to AppURLFacts

Summary

This should work in both dev and prod environments.

Depends on D3547

Test Plan

We tested this migration extensively:

We tested it on the original state I had in my dev environment
We made sure it's idempotent by deleting the migration checkpoint and re-running after the files had been mutated
We also tested it with what we thought we have in prod

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Mar 29 2022, 3:23 PM

Herald added subscribers: • abosh, • benschac, • karol, • adrian. · View Herald TranscriptMar 29 2022, 3:23 PM

ashoat added a child revision: D3549: Revert "[server] add helper function to generate all route paths".Mar 29 2022, 3:23 PM

Harbormaster completed remote builds in B7694: Diff 10796.Mar 29 2022, 3:35 PM

ashoat requested review of this revision.Mar 29 2022, 3:35 PM

varun accepted this revision.Mar 29 2022, 4:45 PM

This revision is now accepted and ready to land.Mar 29 2022, 4:45 PM

Closed by commit rCOMM0122555af744: [server] Introduce migration that adds baseRoutePath to AppURLFacts (authored by ashoat). · Explain WhyMar 30 2022, 11:15 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM0122555af744: [server] Introduce migration that adds baseRoutePath to AppURLFacts.

Revision Contents
Changeset List

Path

Size

server/

src/

database/

migrations.js

34 lines

Diff 10832

View Options

server/src/database/migrations.js

[server] Introduce migration that adds baseRoutePath to AppURLFactsClosedPublicActions