Skip to content

Commit 479543e

Browse files
committed
use github username for my tests link closes #473
1 parent f849a08 commit 479543e

File tree

4 files changed

+3
-6
lines changed

4 files changed

+3
-6
lines changed

server/web/auth/github.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,11 @@ exports.register = function (server, options, next) {
1313
});
1414

1515
// attach credentials to view context to let people know they are logged in
16-
// attach authorGitHub to view context to construct "My Tests" link in footer
1716
server.ext('onPreResponse', function (request, reply) {
1817
const response = request.response;
1918
if (response.variety === 'view') {
2019
response.source.context = response.source.context || {};
2120
response.source.context.credentials = request.auth.isAuthenticated ? request.auth.credentials : null;
22-
response.source.context.authorGitHub = request.yar.get('authorGitHub');
2321
}
2422
return reply.continue();
2523
});

server/web/edit/index.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,6 @@ exports.register = function (server, options, next) {
129129
return pagesService.edit(pageWithTests, update, prevPage.maxRev, prevPage.id);
130130
})
131131
.then(resultingRevision => {
132-
request.yar.set('authorGitHub', pageWithTests.authorGitHub.replace(/[^a-zA-Z0-9-]/, ''));
133132
const r = resultingRevision > 1 ? `/${resultingRevision}` : '';
134133

135134
reply.redirect(`/${request.params.testSlug}${r}`);

server/web/home/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,12 +104,12 @@ exports.register = function (server, options, next) {
104104
slugError: defaults.errors.slugDupe
105105
});
106106
} else {
107+
payload.author = request.auth.credentials.username;
107108
return pagesService.create(payload)
108109
.then(resultPageId => {
109110
const own = request.yar.get('own') || {};
110111
own[resultPageId] = true;
111112
request.yar.set('own', own);
112-
request.yar.set('authorGitHub', payload.authorGitHub.replace(/[^a-zA-Z0-9-]/, ''));
113113
reply.redirect('/' + payload.slug);
114114
});
115115
}

templates/partials/footer.hbs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,9 @@
1616
<a href="/">Add</a>
1717
</li>
1818

19-
{{#if authorGitHub}}
19+
{{#if credentials}}
2020
<li>
21-
<a href="/browse/{{authorGitHub}}">My tests</a>
21+
<a href="/browse/{{credentials.username}}">My tests</a>
2222
</li>
2323
{{/if}}
2424

0 commit comments

Comments
 (0)