- Issue created by @Vighneshh
- Merge request !2Issue-3437871:Added mixpanel token to drupalSettings → (Merged) created by Vighneshh
- Status changed to Needs review
about 1 year ago 7:35pm 4 April 2024 - Status changed to Needs work
about 1 year ago 11:00pm 4 April 2024 - 🇦🇺Australia darvanen Sydney, Australia
I agree this is a good idea, looking good so far, just one comment to address :)
- Status changed to Needs review
about 1 year ago 4:45am 5 April 2024 - 🇦🇺Australia darvanen Sydney, Australia
Looking good, thanks. Doesn't quite adhere to coding standards, if I turned on phpcs I reckon it would complain but at the moment I don't mind with this module.
I'd like feedback from someone else manually testing that this works or even better an automated test.
- Status changed to Fixed
13 days ago 7:04am 24 May 2025 - 🇦🇺Australia darvanen Sydney, Australia
- Added JS test for frontend project token
- Needed to include the core/drupalSettings library in case it is not included anywhere else
- Tidied up the module file a bit to better meet coding standards -
darvanen →
committed 9bf05b0f on 2.x authored by
vighneshh →
Issue #3437871 by vighneshh, darvanen: Mixpanel token should be saved on...
-
darvanen →
committed 9bf05b0f on 2.x authored by
vighneshh →