8000 feat: support for npm by conwnet · Pull Request #410 · conwnet/github1s · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: support for npm #410

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

Merged
merged 3 commits into from
May 28, 2022
Merged

feat: support for npm #410

merged 3 commits into from
May 28, 2022

Conversation

conwnet
Copy link
Owner
@conwnet conwnet commented May 28, 2022

We add support for npm in this PR.
For a preview: https://alpha.npmjs1s.com
also add google analytics code here for basic traffic info.

@vercel
Copy link
vercel bot commented May 28, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
github1s ✅ Ready (Inspect) Visit Preview May 28, 2022 at 5:10PM (UTC)

Copy link
Collaborator
@xcv58 xcv58 left a comment

Choose a reason for hiding this comment

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

This is awesome! lgtm!

@@ -9,7 +9,7 @@
<link rel="icon" media="(prefers-color-scheme:dark)" href="/favicon-dark.svg" type="image/svg+xml" />
<link rel="manifest" href="/manifest.json" />
<title>GitHub1s</title>
<script>if (window.location.hostname.match(/\.?gitlab1s\.com$/i)) { window.document.title = 'GitLab1s'; } else if (window.location.hostname.match(/\.?bitbucket1s\.org$/i)) { window.document.title = 'Bitbucket1s'; };</script>
<script>if (window.location.hostname.match(/\.?gitlab1s\.com$/i)) { window.document.title = 'GitLab1s'; } else if (window.location.hostname.match(/\.?bitbucket1s\.org$/i)) { window.document.title = 'Bitbucket1s'; } else if (window.location.hostname.match(/\.?npmjs1s\.com$/i)) { window.document.title = 'npm1s'; };</script>
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we can refactor this to make it reusable in all .html files without copy paste, I will look into it.

Copy link
Owner Author

Choose a reason for hiding this comment

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

yes! thank you for your advice, i think we can migrate these to the build scripts

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm working on this, will share a PR soon.

@conwnet conwnet merged commit a5748f8 into master May 28, 2022
@conwnet conwnet deleted the npmjs1s branch May 28, 2022 21:12
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.

2 participants
0