Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

clarified comment for author.stackoverflow value in _config.yml #487

Closed
wants to merge 1 commit into from
Closed

clarified comment for author.stackoverflow value in _config.yml #487

wants to merge 1 commit into from

Conversation

rschaerer
Copy link
Contributor

Initial Situation

  • The author.stackoverflow variable was commentet with # http://stackoverflow.com/users/123456/username, which led me to the conclusion that I should add the whole url as value.
  • The stackoverflow combination of numeric id / username isn't self-explanatory.

Proposed Change

Change the comment to explicit state the requested value "123456/username" and append the whole demo url to show where the needed value can be found.

@mmistakes
Copy link
Owner

Thanks. Merged into develop and will push with the next theme release.

ref: d3ac112

@mmistakes mmistakes closed this Aug 30, 2016
@rschaerer rschaerer deleted the clarified-comment-stackoverflow-author-profile branch September 28, 2016 22:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants