Page MenuHomePhabricator
Differential D3551

[server] Use separate Express router for landing page
ClosedPublic
Actions

Authored by ashoat on Mar 29 2022, 3:25 PM.
Tags
None
Referenced Files
F5096722: D3551.diff
Sun, Mar 30, 9:18 PM
F5093785: D3551.id10799.diff
Sun, Mar 30, 1:24 PM
Unknown Object (File)
Thu, Mar 13, 2:35 PM
Unknown Object (File)
Thu, Mar 13, 2:35 PM
Unknown Object (File)
Thu, Mar 13, 2:35 PM
Unknown Object (File)
Thu, Mar 13, 2:31 PM
Unknown Object (File)
Tue, Mar 4, 2:20 PM
Unknown Object (File)
Feb 23 2025, 5:07 PM
View All Files
Subscribers
• abosh
• adrian
• benschac
• karol

Details

Reviewers
varun
atul
tomek
Commits
rCOMM36d3c9543a04: [server] Use separate Express router for landing page
Summary

This diff also makes us no longer needs server/facts/url.json. The baseRoutePath that was there was ostensibly a global one, but if we have distinct routers for each "app", we don't need a global baseRoutePath anymore.

Depends on D3550

Test Plan
  1. We tested the landing page, the SquadCal app, and the Comm app
  2. We made sure we tried configs where baseRoutePath !== basePath
  3. We also checked view-source to make sure things were set correctly
  4. We made sure there were no errors in the browser console
  5. We played around with the site, logging in and navigating and making sure things worked correctly
  6. We tried loading a specific URL for a specific chat directly

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ashoat created this revision.Mar 29 2022, 3:25 PM
Herald added subscribers: • abosh, • benschac, • karol, • adrian. · View Herald TranscriptMar 29 2022, 3:25 PM
ashoat added a child revision: D3552: [server] Use same Router config for web.comm.app that we use for squadcal.org.Mar 29 2022, 3:26 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Mar 29 2022, 3:35 PM
Harbormaster failed remote builds in B7697: Diff 10799!
Harbormaster completed remote builds in B7697: Diff 10799.Mar 30 2022, 10:57 AM
ashoat requested review of this revision.Mar 30 2022, 10:57 AM
varun accepted this revision.Mar 30 2022, 11:01 AM
varun added inline comments.
server/src/database/migrations.js
44–46 ↗(On Diff #10799)

unlink is wrapped in a try-catch so that the migration is idempotent

This revision is now accepted and ready to land.Mar 30 2022, 11:01 AM
Closed by commit rCOMM36d3c9543a04: [server] Use separate Express router for landing page (authored by ashoat). · Explain WhyMar 30 2022, 11:15 AM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMM36d3c9543a04: [server] Use separate Express router for landing page.

Revision Contents
Changeset List

PathSize
server/
src/
database/
8 lines
60 lines
utils/
10 lines

Diff 10835

View Options

server/src/database/migrations.js

div.membersContainer {
display: flex;
flex-direction: column;
overflow: hidden;
margin: 16px;
}
div.membersList { div.membersList {
overflow: scroll; overflow: scroll;
padding: 8px 0; padding: 8px 0;
color: var(--members-modal-member-text); color: var(--members-modal-member-text);
} }
div.noScroll { div.noScroll {
overflow: hidden; overflow: hidden;
Show All 38 Lines
View Options

server/src/server.js

div.membersContainer {
display: flex;
flex-direction: column;
overflow: hidden;
margin: 16px;
}
div.membersList { div.membersList {
overflow: scroll; overflow: scroll;
padding: 8px 0; padding: 8px 0;
color: var(--members-modal-member-text); color: var(--members-modal-member-text);
} }
div.noScroll { div.noScroll {
overflow: hidden; overflow: hidden;
Show All 38 Lines
View Options

server/src/utils/urls.js

div.membersContainer {
display: flex;
flex-direction: column;
overflow: hidden;
margin: 16px;
}
div.membersList { div.membersList {
overflow: scroll; overflow: scroll;
padding: 8px 0; padding: 8px 0;
color: var(--members-modal-member-text); color: var(--members-modal-member-text);
} }
div.noScroll { div.noScroll {
overflow: hidden; overflow: hidden;
Show All 38 Lines