Skip to content

added feature for user agent#3376

Open
SwarajG wants to merge 8 commits intopear-devs:masterfrom
SwarajG:user-agent-feature
Open

added feature for user agent#3376
SwarajG wants to merge 8 commits intopear-devs:masterfrom
SwarajG:user-agent-feature

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
Member

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
Member

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