Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Docs: Error fixes & Improvements #248

Open
wants to merge 8 commits into
base: gh-pages
Choose a base branch
from

Conversation

akhileshthite
Copy link
Contributor

@akhileshthite akhileshthite commented May 22, 2021

Description

• Added hyperlink to dogecoin.com for quick access to the website, as there is no website link on the readme and in the repo description.
• Fixed some minor grammatical errors.

Checklist:

  • Any documentation has been updated accordingly

aarsxx
aarsxx previously approved these changes May 23, 2021
Copy link

@amirarsalanpakravan amirarsalanpakravan left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link

@moritzebeling moritzebeling left a comment

Choose a reason for hiding this comment

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

Looks good. I just think it would be nice to keep Pls instead of Please

README.md Outdated
@@ -11,7 +11,7 @@ Designed to retain the spirit of Dogecoin, whilst appealing to the more serious
Dogecoin.com is completely responsive, it'll look good on your smartphone, tablet, desktop, and more.
It is completely HTML5 W3C Validated, which means it complies with the standards set by W3C.

## Technical Details Pls.
## Technical Details Please.
Copy link

@moritzebeling moritzebeling May 25, 2021

Choose a reason for hiding this comment

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

I suggest to keep "Pls" to keep the doge tone intact.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done sir :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants