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

fix: patch d2-analysis so elements have relevant class-names for push-analytics #1032

Merged
merged 3 commits into from
Jan 29, 2024

Conversation

HendrikThePendric
Copy link
Contributor

This PR upgrades d2-analysis. The new version comes with some hardcoded class-names that can be used by push-analysis.

@HendrikThePendric
Copy link
Contributor Author

@janhenrikoverland as discussed, this now doesn't upgrade d2-analysis but patches it instead. I also ran npx yarn-deduplicate, which explains a lot of the changes in yarn.lock.

@HendrikThePendric HendrikThePendric changed the title fix: upgrade d2-analysis to v33.3.0 fix: patch d2-analysis so elements have relevant class-names for push-analytics Jan 29, 2024
@HendrikThePendric HendrikThePendric merged commit a844a6b into master Jan 29, 2024
4 checks passed
@HendrikThePendric HendrikThePendric deleted the fix/upgrade-d2-analysis branch January 29, 2024 14:02
dhis2-bot added a commit that referenced this pull request Jan 29, 2024
## [33.0.56](v33.0.55...v33.0.56) (2024-01-29)

### Bug Fixes

* patch d2-analysis so elements have relevant class-names for push-analytics ([#1032](#1032)) ([a844a6b](a844a6b))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 33.0.56 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants