-
-
Notifications
You must be signed in to change notification settings - Fork 775
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
Fixed Gitcoinbot Documentation Content #5607
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5607 +/- ##
=========================================
Coverage ? 29.57%
=========================================
Files ? 266
Lines ? 22653
Branches ? 3289
=========================================
Hits ? 6700
Misses ? 15676
Partials ? 277 Continue to review full report at Codecov.
|
@octavioamu @thelostone-mc Take a look.. |
No active bounty for this issue, consider create the bounty please visit this link. PS Make sure you're logged into Metamask! |
@thelostone-mc Review my friend |
@danlipert Updated only content, leave the rest to mkdocs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @molecula451 - just a few fixes and this should be good to merge. Also, please don't spam the core engineers with review requests, we automatically get notified each time a commit is added or a PR is marked ready for review.
@danlipert Can we end up this? can you please double-check all the most recent fixes? |
@danlipert Up to you... |
@danlipert I will be happy if you finally review this asap so I can clean up my repo |
Description
Our Section for GitcoinBot in the Documentation has some broken gif links which are intended to show our Bot in Action. We made some changes and update images so our users can actually see our examples.
Actual Behaviour:
Refers/Fixes
Testing
Chrome
Brave Browser