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

Removed Reporting options from the WebUI and API #162

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 0 additions & 3 deletions frontend/src/API.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,6 @@ class Api {

async core() {
const core = axios.get('api').then(response => (response.data))
if (core.allow_reports) {
await this.sentry_init()
}
return core
}

Expand Down
13 changes: 0 additions & 13 deletions frontend/src/forms/CoreSettings.vue
Original file line number Diff line number Diff line change
Expand Up @@ -50,19 +50,6 @@
</select>
</div>

<div class="form-group row mt-3">
<label class="col-sm-10 col-form-label">{{ $t('send_reports') }}</label>
<div class="col-sm-2 float-right">
<span @click="core.allow_reports = !!core.allow_reports" class="switch" id="allow_report">
<input v-model="core.allow_reports" type="checkbox" name="allow_report" class="switch" id="switch_allow_report" :checked="core.allow_reports">
<label for="switch_allow_report"></label>
</span>
</div>
<div class="col-12">
<small>{{ $t('send_reports_desc') }}</small>
</div>
</div>

</div>
<div class="card-footer">
<button @click.prevent="saveSettings" id="save_core" type="submit" class="btn btn-primary btn-block" v-bind:disabled="loading">
Expand Down
25 changes: 3 additions & 22 deletions frontend/src/forms/Setup.vue
Original file line number Diff line number Diff line change
Expand Up @@ -58,19 +58,6 @@
<input @keyup="canSubmit" v-model="setup.db_database" id="db_database" type="text" class="form-control" placeholder="Database name">
</div>

<div class="form-group mt-3">
<div class="row">
<div class="col-9">
<span class="text-left text-capitalize">{{ $t('send_reports') }}</span>
</div>
<div class="col-3 text-right">
<span @click="setup.send_reports = !!setup.send_reports" class="switch">
<input v-model="setup.send_reports" type="checkbox" name="send_reports" class="switch" id="send_reports" :checked="setup.send_reports">
<label for="send_reports"></label>
</span>
</div>
</div>
</div>

</div>

Expand Down Expand Up @@ -113,13 +100,7 @@
<label class="text-capitalize">{{ $t('email') }}</label>
<input @keyup="canSubmit" v-model="setup.email" id="email" type="text" class="form-control" placeholder="myemail@domain.com">
</div>
<div class="col-4 text-right">
<label class="d-none d-sm-block text-capitalize text-capitalize">{{ $t('newsletter') }}</label>
<span @click="setup.newsletter = !!setup.newsletter" class="switch">
<input v-model="setup.newsletter" type="checkbox" name="send_newsletter" class="switch" id="send_newsletter" :checked="setup.newsletter">
<label for="send_newsletter"></label>
</span>
</div>
<div class="col-4 text-right">&nbsp;</div>
</div>
<small>{{ $t('newsletter_note') }}</small>
</div>
Expand Down Expand Up @@ -167,8 +148,8 @@
password: "",
confirm_password: "",
sample_data: true,
newsletter: true,
send_reports: true,
newsletter: false,
send_reports: false,
email: "",
}
}
Expand Down
2 changes: 1 addition & 1 deletion frontend/src/pages/Help.vue
Original file line number Diff line number Diff line change
Expand Up @@ -435,7 +435,7 @@ services:
<li><code>GO_ENV</code> - Run Statping in testmode, will bypass HTTP authentication (if set as &lsquo;test&rsquo;)</li>
<li><code>REMOVE_AFTER</code> - Automatically delete records after time (default 3 months, &lsquo;12h = 12 hours&rsquo;)</li>
<li><code>CLEANUP_INTERVAL</code> - Interval to check for old records (default 1 hour, &lsquo;1h = 1 hour&rsquo;)</li>
<li><code>ALLOW_REPORTS</code> - Send Statping anonymous <a href="https://sentry.io/" target="_blank">error reports</a> so we can see issues (default is false)</li>
<li><code>ALLOW_REPORTS</code> - Send Statping anonymous <a href="https://sentry.io/" target="_blank">error reports</a> so we can see issues (default is false) - Should be removed in Statping-ng</li>
<li><code>SERVER_PORT</code> - Port number to run Statping HTTP server on (or use -p/&ndash;port)</li>
</ul>

Expand Down