Skip to content
This repository has been archived by the owner on Apr 3, 2019. It is now read-only.

feat(avatars): enable default avatar #304

Merged
merged 2 commits into from
Feb 26, 2018
Merged

feat(avatars): enable default avatar #304

merged 2 commits into from
Feb 26, 2018

Conversation

vladikoff
Copy link
Contributor

Fixes #295

@ghost ghost assigned vladikoff Feb 23, 2018
@ghost ghost added the waffle:active label Feb 23, 2018
@vladikoff vladikoff requested a review from rfk February 24, 2018 15:14
@vladikoff
Copy link
Contributor Author

@rfk r?

Added information into the deploy doc after talking this through with @jrgm

Copy link
Contributor

@rfk rfk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

docs/API.md Outdated

#### Response

> Note: we no longer support Gravatar uploading, but there are entries in the database
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are there? Side note, but I wonder if we should somehow purge them.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mistake, we actually used to process them, I guess the docs always had an incorrect url

@@ -20,6 +28,26 @@ exports.create = function() {
port: config.server.port + 1
});

server.route({
method: 'GET',
path: '/a/' + DEFAULT_AVATAR_ID + '{type?}',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 I like this way of just making it all a bit simpler for dev

@vladikoff vladikoff merged commit 01b0e41 into master Feb 26, 2018
@vladikoff vladikoff deleted the avatar-default-2 branch February 26, 2018 19:29
@vladikoff vladikoff restored the avatar-default-2 branch February 28, 2018 03:44
vladikoff added a commit that referenced this pull request Feb 28, 2018
vladikoff added a commit that referenced this pull request Feb 28, 2018
vladikoff added a commit that referenced this pull request Feb 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants