Skip to content

Commit e9602ca

Browse files
committed
Do not log express URL to console
The review app’s Express instance runs on port 8080, and then we run Browsersync on port 3000 which is the primary port we expect users to use when using the review app. However at the minute we log the Express URL last in the terminal: ``` [app] [Browsersync] Access URLs: [app] --------------------------------------- [app] Local: http://localhost:3000 [app] External: http://192.168.68.119:3000 [app] --------------------------------------- [app] UI: http://localhost:3001 [app] UI External: http://192.168.68.119:3001 [app] --------------------------------------- [app] [Browsersync] Watching files... [app] Server started at http://localhost:8080 ``` We’re showing the user 5 different URLs here, and putting the onus on them to work out which one they need to use. Reduce the confusion by not logging the Express URL, and only outputting the BrowerSync URLs.
1 parent 0d98d82 commit e9602ca

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed
Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,7 @@
1-
import { ports, urls } from '@govuk-frontend/config'
1+
import { ports } from '@govuk-frontend/config'
22

33
import app from './app.mjs'
44

55
const server = await app()
66

7-
server.listen(ports.app, () => {
8-
console.log(`Server started at ${urls.app}`)
9-
})
7+
server.listen(ports.app)

0 commit comments

Comments
 (0)