Skip to content

Commit

Permalink
update matomo modal (#941)
Browse files Browse the repository at this point in the history
before merging - please follow the link to our matomo instance to make sure it is showing relevant info.  Also there is not a link to the medium article (which would be better) nor to the docs - so the info we are collecting is not specified.
  • Loading branch information
ryestew committed Mar 4, 2021
1 parent 934f491 commit 58ef7a7
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
10 changes: 6 additions & 4 deletions apps/remix-ide/src/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -368,12 +368,14 @@ Please make a backup of your contracts and start using http://remix.ethereum.org
// Ask to opt in to Matomo for remix, remix-alpha and remix-beta
if (window.location.hostname.includes('.ethereum.org') && !registry.get('config').api.exists('settings/matomo-analytics')) {
modalDialog(
'Help us to improve our IDE!',
'Help us to improve our IDE',
yo`
<div>
<p>Remix IDE uses <a href="https://matomo.org">Matomo</a>, an open source data analytics software, to improve the use of our website.</p>
<p>We realize that our users have sensitive information in their code and that the privacy of our users must be protected, therefor we do not store any personally identifiable information (PII).</p>
<p>All data collected through Matomo is stored at our own server - no data is given to third parties.</p>
<p>Remix IDE uses <a href="https://matomo.org target="_blank">Matomo</a>, an open source data analytics platform, to improve our website. Matomo on Remix is opt-in - meaning that we won't collect any information unless you agree.</p>
<p>We realize that our users have sensitive information in their code and that the privacy of our users must be protected.</p>
<p>All data collected through Matomo is stored on our own server - no data is ever given to third parties. Our analytics reports are public: <a href="https://matomo.ethereum.org/index.php?module=MultiSites&action=index&idSite=23&period=day&date=yesterday" target="_blank">take a look</a>.</p>
<p>We do not store any personally identifiable information (PII).</p>
<p>For more info see: <a href="https://medium.com/p/66ef69e14931/" target="_blank">Matomo Analyitcs on Remix iDE</a>.</p>
<p>You can change your choice in the Settings panel anytime.</p>
<div class="d-flex justify-content-around pt-3 border-top">
<button class="btn btn-primary ${css.matomoBtn}" onclick=${() => onAcceptMatomo()}>Sure</button>
Expand Down
3 changes: 1 addition & 2 deletions apps/remix-ide/src/app/tabs/settings-tab.js
Original file line number Diff line number Diff line change
Expand Up @@ -125,8 +125,7 @@ module.exports = class SettingsTab extends ViewPlugin {
this._view.useMatomoAnalyticsLabel = yo`
<label class="form-check-label custom-control-label align-middle" for="settingsMatomoAnalytics">
<span>Enable Matomo Analytics. We do not collect personally identifiable information (PII). The info is used to improve the site’s UX & UI. See more about</span>
<a href="https://remix-ide.readthedocs.io/en/latest/FAQ.html#analytics" target="_blank">Analytics in Remix IDE</a> <span>&</span> <a target="_blank" href="https://matomo.org/free-software">Matomo</a>
<a href="https://medium.com/p/66ef69e14931/" target="_blank">Analytics in Remix IDE</a> <span>&</span> <a target="_blank" href="https://matomo.org/free-software">Matomo</a>
</label>
`
if (this.config.get('settings/matomo-analytics')) {
Expand Down

0 comments on commit 58ef7a7

Please sign in to comment.