Skip to content

Allow setting form factors client hint#136

Merged
sgiehl merged 4 commits into
3.x-devfrom
formfactors
Oct 9, 2024
Merged

Allow setting form factors client hint#136
sgiehl merged 4 commits into
3.x-devfrom
formfactors

Conversation

@sgiehl

@sgiehl sgiehl commented Sep 3, 2024

Copy link
Copy Markdown
Member

Description:

refs matomo-org/matomo#22549

Review

@sgiehl
sgiehl force-pushed the formfactors branch 2 times, most recently from 7507825 to 929a9b1 Compare September 3, 2024 09:35
@sgiehl sgiehl changed the title Allow settting form factors client hint Allow setting form factors client hint Oct 8, 2024
@sgiehl
sgiehl changed the base branch from master to 3.x-dev October 8, 2024 13:20
@sgiehl
sgiehl marked this pull request as ready for review October 8, 2024 16:34
@sgiehl
sgiehl requested a review from a team October 8, 2024 16:37
Comment thread MatomoTracker.php Outdated
Co-authored-by: Michal Kleiner <mk@011.nz>
@sgiehl
sgiehl merged commit 949259d into 3.x-dev Oct 9, 2024
@sgiehl
sgiehl deleted the formfactors branch October 9, 2024 08:10
sgiehl added a commit that referenced this pull request Dec 20, 2025
* Allow settting form factors client hint

* update changelog

* Fix handling of some attributes

* Apply review feedback

Co-authored-by: Michal Kleiner <mk@011.nz>

---------

Co-authored-by: Michal Kleiner <mk@011.nz>
sgiehl added a commit that referenced this pull request Dec 20, 2025
…ifferent API urls (#145)

* Allow setting form factors client hint (#136)

* Allow settting form factors client hint

* update changelog

* Fix handling of some attributes

* Apply review feedback

Co-authored-by: Michal Kleiner <mk@011.nz>

---------

Co-authored-by: Michal Kleiner <mk@011.nz>

* deprecate $URL

* use old $URL if apiUrl is empty

---------

Co-authored-by: Stefan Giehl <stefan@matomo.org>
Co-authored-by: Michal Kleiner <mk@011.nz>
Co-authored-by: Andrii Lutskevych <andrii.lutskevych@check24.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants