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

Provide an option to remove "errors" tab #88

Closed
ColtonIdle opened this issue Aug 10, 2019 · 12 comments
Closed

Provide an option to remove "errors" tab #88

ColtonIdle opened this issue Aug 10, 2019 · 12 comments
Labels
feature request Request a new feature to be developed help wanted Extra attention is needed

Comments

@ColtonIdle
Copy link
Sponsor

I use a few other crash reporters and crash viewers built into my application. The fact that Chucker added this onto Chuck is nice, but I want to get rid of it so my team doesn't get confused about whether or not we should be using it vs our other error reporters. During Chucker setup, it'd be nice to just opt-out of having the error feature showing/working.

@cortinico
Copy link
Member

@ColtonIdle
Are you suggesting a way to hide the "Errors" tab right?

@ColtonIdle
Copy link
Sponsor Author

Yep. Basically to only have the original functionality of chuck. Which is to inspect network traffic because I already use other tools for errors.

@cortinico
Copy link
Member

Yep. Basically to only have the original functionality of chuck. Which is to inspect network traffic because I already use other tools for errors.

I see. As a side note: Chucker is not capturing crashes unless you explicitly call collector.onError(). So the Error tab won't be populated automatically.

@cortinico cortinico added feature request Request a new feature to be developed help wanted Extra attention is needed labels Aug 11, 2019
@uOOOO
Copy link
Contributor

uOOOO commented Aug 19, 2019

I'll create PR after #98 is merged.

@vbuberen
Copy link
Collaborator

vbuberen commented Apr 6, 2020

Would like to return to this question. I haven't seen much feedback or requests about error reporting feature since I joined the team, so it just gives some points to my suggestion of removing this functionality at all and making Chucker focused on network only.

I believe that it is better to focus on something specific in such tool and do it great.
So I suggest to remove Errors tab at all.

@cortinico
Copy link
Member

I'm also in for removing it. Maybe we want to loop in with @olivierperez since he initially developed it. Are you actually using this feature? I would love to understand what's the adoption rate of it.

@olivierperez
Copy link
Member

Hi, I'm ok too. This feature seems to be unused.
Can we at least make a poll on Twitter to ask people if the want to keep it or drop it?

@vbuberen
Copy link
Collaborator

vbuberen commented Apr 7, 2020

Pool is a great idea. Let's do it :)

@cortinico
Copy link
Member

I'd rather create a poll as a Github Issue as the majority of our users are there. We can set the issue as sticky and have a deadline say 2/3 weeks. People can vote using reactions like:

👍 Let's keep the feature
👎 Let's remove the feature
👀 See comment below

@vbuberen
Copy link
Collaborator

vbuberen commented Apr 8, 2020

@cortinico Ok, I will create such issue. But I also would like to ask you to also post the link to tbis issue in Twitter, since your account is the most popular among us.

@cortinico
Copy link
Member

Sure let's do that 👍

@vbuberen
Copy link
Collaborator

Closing this one due to deprecation of Errors tab

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Request a new feature to be developed help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants