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: added carbon ad #1337

Merged
merged 2 commits into from
Aug 17, 2020
Merged

Docs: added carbon ad #1337

merged 2 commits into from
Aug 17, 2020

Conversation

anikethsaha
Copy link
Member

Added CarbonAds

@vercel
Copy link

vercel bot commented Aug 16, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/docsify-core/docsify-preview/4wtw3y61l
✅ Preview: https://docsify-preview-git-carbonads.docsify-core.vercel.app

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 16, 2020

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 450189d:

Sandbox Source
docsify-template Configuration

docs/index.html Outdated Show resolved Hide resolved
@anikethsaha anikethsaha added priority : high website related to the official docsify website: https://docsify.js.org labels Aug 16, 2020
Copy link
Member

@Koooooo-7 Koooooo-7 left a comment

Choose a reason for hiding this comment

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

@sy-records I agree we can use that ads plugin or we can write the style instead of importing extra resource.

@anikethsaha
Copy link
Member Author

agreed let's use the plugin

@anikethsaha
Copy link
Member Author

We will need to release a version once it is merged in develop, I can directly submit a PR against master as there would be conflicts.

Copy link
Member

@Koooooo-7 Koooooo-7 left a comment

Choose a reason for hiding this comment

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

I m just working on it also, it seems a litte big.
LGTM.

@anikethsaha
Copy link
Member Author

I m just working on it also, it seems a litte big.

yeah if PR against master then the diff is quite big, We need to release it at the earliest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority : high website related to the official docsify website: https://docsify.js.org
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants