Skip to content

added title to settings page #7592

Closed
wants to merge 1 commit into from

3 participants

@AkiraLaine
Free Code Camp member

Can someone check it out and make sure it works? My version of FCC can't run because I can't seem to install a dependency.

@alistermada

Yep, it works. :+1:

image

@AkiraLaine
Free Code Camp member

@alistermada Awesome! Thanks.

@raisedadead raisedadead commented on the diff
server/boot/user.js
@@ -249,7 +249,9 @@ module.exports = function(app) {
}
function getSettings(req, res) {
- res.render('account/settings');
+ res.render('account/settings', {
+ title: 'Settings | Free Code Camp'
@raisedadead
Free Code Camp member

Change title: 'Settings | Free Code Camp' to title: 'Settings '

The tool tip shows an extra | Free Code Camp :

1

@raisedadead
Free Code Camp member

The changed one will show:

2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@raisedadead raisedadead added the blocked label
@raisedadead
Free Code Camp member

Closing in favor of #7601

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.