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

Remove tracking tools #273

Merged
merged 2 commits into from
Oct 8, 2024
Merged

Remove tracking tools #273

merged 2 commits into from
Oct 8, 2024

Conversation

Cipulot
Copy link
Contributor

@Cipulot Cipulot commented Oct 8, 2024

As per title.

@Cipulot Cipulot requested a review from olivia October 8, 2024 21:15
@Cipulot
Copy link
Contributor Author

Cipulot commented Oct 8, 2024

This should address 226 and 244.

Copy link
Contributor

@olivia olivia left a comment

Choose a reason for hiding this comment

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

We should investigate something like https://docs.sentry.io/platforms/javascript/user-feedback/#crash-report-modal to not completely remove any mechanism for a user to report errors, but that’s not a blocker for this.

@Cipulot
Copy link
Contributor Author

Cipulot commented Oct 8, 2024

We should investigate something like https://docs.sentry.io/platforms/javascript/user-feedback/#crash-report-modal to not completely remove any mechanism for a user to report errors, but that’s not a blocker for this.

I think for now removing them all together is ok, and in the meantime explore alternative methods of collection in case user wants to share info.

@Cipulot Cipulot merged commit 88d4b9c into the-via:main Oct 8, 2024
1 check passed
@Cipulot Cipulot deleted the remove_stuff branch October 8, 2024 22:39
@Niko-O
Copy link

Niko-O commented Oct 8, 2024

When can we expect this change to be in effect?

@Cipulot
Copy link
Contributor Author

Cipulot commented Oct 11, 2024

@Niko-O it should be live already. From what I recall the deploy happens as soon as GH actions in this repo are done.

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