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

Added query params, cleaned code & UI #12

Closed
wants to merge 1 commit into from

Conversation

RyfterWasTaken
Copy link

@RyfterWasTaken RyfterWasTaken commented Mar 15, 2024

Accidentally deleted the old one, I'll keep this PR open in case someone wants to use it,

You can use this link to deploy it to vercel, its the same steps as deploying it normally

1. Added optional query parameters:

  • red=link will redirect to such link once clicked on
  • img=image link will show the image instead of the pixel

2. Added UA:

  • Noticed that it wouldnt load images without another ua set to return the image, so it would bug
  • Added vercelbot ua

3. Cleaned code&UI:

  • Modified multiple code parts, such as using req.nextUrl.pathname instead of req.url.slice("/").
  • Modified the embed to send fields, instead of footer& description

4. Silenced Vercel & /favicon hits

Accidentally deleted the old one
Copy link

vercel bot commented Mar 15, 2024

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

Name Status Preview Comments Updated (UTC)
view-loggers ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 15, 2024 1:33pm

@RyfterWasTaken
Copy link
Author

RyfterWasTaken commented Mar 15, 2024

compacted it and cleared it a bit, and changed ifs to single line, and added vercel redirection to /vercel.html

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.

1 participant