`clean-conversation-sidebar` is broken #4238
SimonBrandner posted onGitHub
Extension enabled:
Extension disabled:
Extension version: 21.4.13 FF version: 78.9.0esr
Note: Sorry if I am missing something here :sweat_smile:
Ah, solution is to disable this:
But still seems like a bug to me. At least if I have triage access to the repo at question, I don't expect that to be hidden. Please correct me if I am wrong though
Hmm looking at the code I don't think that should happen.
Are you able to copy the DOM here?
Here you go
Thanks
I dont see the .discussion-sidebar-item
selector there π
Can you do $('.discussion-sidebar-item') in the console (with the ext off) and copy that elements DOM?
Here. Thanks for looking into this, btw!
Did not fully debug but it looks like the selector here maybe .sidebar-assignee
Maybe this has something to do with how we detect "can user edit repo" since it's a "all or nothing" check. As you can see, you Projects section does not have a gear icon, but collaborators do: <img width="310" alt="Screen Shot" src="https://user-images.githubusercontent.com/1402241/114746457-fb70a800-9d79-11eb-8b7b-4e37c0a7d0df.png">
I donβt think it's enterprise only. Triage is a type of collaborator
I would try to fix it. Do you have a repo you can add me on to debug?
@fregante can you reenable and disable via hotfix?
I'm not sure I understand
We commented out the line
What's the situation on this feature? Is it still completely broken? I think there's no difference between how it is now and "hotfixing" it
Correct but hotfixing it will allow users to know it was disabled and why.
Thank you!