Skip to content
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

Release branch google analytics #130

Conversation

JazzarKarim
Copy link
Collaborator

@JazzarKarim JazzarKarim commented Jan 23, 2025

*Issue:*bcgov/entity#25401

Description of changes:
This is just a release branch created from commit of version 1.0.5. The version here is "version": "1.0.5b"

The idea is we want those google analytics ASAP. This is the exact same commit as the latest one of mine in main.

Keep in mind, I'm merging into that release branch, not main.

I cherry picked that google analytics commit from main.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

JazzarKarim and others added 2 commits January 23, 2025 11:49
* 25401 - Added google analytics support

* committing the lock file

* fixed lint

* trying to fix unit tests

* trying to fix unit tests

* reverted setup file change

* vitest update

* updated the github ci file

* fixed cypress tests

* updated package version
Copy link
Collaborator

@severinbeauvais severinbeauvais left a comment

Choose a reason for hiding this comment

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

LGTM.

I think you'll have to manually deploy this branch into Dev (and then restore Dev later).

@JazzarKarim JazzarKarim merged commit 97fd883 into bcgov:google-analytics-release-branch Jan 23, 2025
5 checks passed
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.

2 participants