Skip to content

Conversation

@DeckerSU
Copy link
Contributor

@DeckerSU DeckerSU commented Oct 2, 2025

Updated the build script to handle Matomo tracking configuration:

  • Requires MATOMO_URL and MATOMO_SITE_ID environment variables.
  • Injects both values into BUILD_CMD via --dart-define when present.
  • Show a warning message if one or both parameters are missing.

This ensures builds only proceed when Matomo tracking is properly configured.

Updated the build script to handle Matomo tracking configuration:

- Requires MATOMO_URL and MATOMO_SITE_ID environment variables.
- Injects both values into BUILD_CMD via --dart-define when present.
- Show a warning message if one or both parameters are missing.

This ensures builds only proceed when Matomo tracking is properly configured.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 2, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch patch-add-matomo-docker

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@DeckerSU DeckerSU requested review from CharlVS and smk762 October 2, 2025 17:11
@github-actions
Copy link

github-actions bot commented Oct 2, 2025

Visit the preview URL for this PR (updated for commit 4ea5b60):

https://walletrc--pull-3165-merge-y7xj0dbf.web.app

(expires Thu, 09 Oct 2025 17:15:15 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: f66a4ff03faa546f12f0ae5a841bd9eff2714dcc

@CharlVS CharlVS merged commit a6afdf0 into dev Oct 2, 2025
9 of 15 checks passed
@CharlVS CharlVS deleted the patch-add-matomo-docker branch October 2, 2025 17:23
@CharlVS CharlVS mentioned this pull request Oct 5, 2025
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.

3 participants