Skip to content
This repository has been archived by the owner on Dec 15, 2020. It is now read-only.

Star link got fixed #841

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Star link got fixed #841

wants to merge 5 commits into from

Conversation

SMAKSS
Copy link

@SMAKSS SMAKSS commented Oct 24, 2020

Fixes #781 and closing (PR) #810 which do not fix this problem.

  • Explain the motivation for making this change.
  • Provide a test plan demonstrating that the code is solid.
  • Match the code formatting of the rest of the codebase.

Motivation (required)

What existing problem does the pull request solve?

siteConfig.js includes a property named as repoUrl which is commented earlier, this property was used as a href for star link in the footer, so it caused it to not work as expected.

Test Plan (required)

You can simply try the star link in the footer to see if it works or not.

star

Fixing grammar and snippet languages in the `docs/*.md` files.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Star link in Footer isn't working [Website]
2 participants