Skip to content

Browscap mapping#418

Open
bidi47 wants to merge 5 commits into7.0from
browscap
Open

Browscap mapping#418
bidi47 wants to merge 5 commits into7.0from
browscap

Conversation

@bidi47
Copy link
Copy Markdown
Member

@bidi47 bidi47 commented Apr 17, 2026

No description provided.

@bidi47 bidi47 linked an issue Apr 17, 2026 that may be closed by this pull request
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 17, 2026

Qodana for JS

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@bidi47 bidi47 requested review from alexmerlin and arhimede April 17, 2026 10:35
Comment thread src/Admin/src/Service/AdminLoginService.php Outdated
/**
* For device information
*
* @see https://github.com/dotkernel/dot-user-agent-sniffer
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this comment need to be updated
to mention the installation of browcap ( link to php manual )

@arhimede
Copy link
Copy Markdown
Member

@bidi47 can you fix the missing DCO ?

@arhimede
Copy link
Copy Markdown
Member

I would like to see a test to cover this ?
I am asking too mucho ?

bidi47 added 4 commits April 17, 2026 15:43
Signed-off-by: bidi <bidi@apidemia.com>
Signed-off-by: bidi <bidi@apidemia.com>
Signed-off-by: bidi <bidi@apidemia.com>
Signed-off-by: bidi <bidi@apidemia.com>
Signed-off-by: bidi <bidi@apidemia.com>
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.

implement browsecap

2 participants