Skip to content

added feature for user agent #3376

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

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

Conversation

SwarajG
Copy link

@SwarajG SwarajG commented May 17, 2025

  • Added feature for user agent
    • User can add a new user agent
    • User can reset user agent

@ArjixWasTaken

This comment has been minimized.

@SwarajG
Copy link
Author

SwarajG commented May 19, 2025

Removing translation, used AI for that.

@SwarajG
Copy link
Author

SwarajG commented May 19, 2025

Reverted language changes. @ArjixWasTaken

@SwarajG
Copy link
Author

SwarajG commented May 20, 2025

I have added this package. "ua-parser-js": "2.0.3" to parse the user agent string.
Looks like, package.json does not have the same package and that's why it is failing.
What should be the next steps?

@SwarajG
Copy link
Author

SwarajG commented May 22, 2025

Can someone please help here?

@SwarajG
Copy link
Author

SwarajG commented May 26, 2025

@ArjixWasTaken Please help me on the next steps

@ArjixWasTaken
Copy link
Contributor

You need to run pnpm i to fix the pnpm-lock.yaml file

@SwarajG
Copy link
Author

SwarajG commented May 30, 2025

Done with the changes

@SwarajG
Copy link
Author

SwarajG commented Jun 2, 2025

@ArjixWasTaken Please merge or help me with the next step.

@ArjixWasTaken
Copy link
Contributor

I am a normal contributor just like you, I just try to help other contributors in my limited free time, so I can't merge this PR!

But, in my opinion this PR LGTM!

@SwarajG
Copy link
Author

SwarajG commented Jun 2, 2025

@JellyBrick Can you please merge this?

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.

3 participants