sindresorhus/refined-github

`clean-conversation-sidebar` is broken #4238

SimonBrandner posted onGitHub

Extension enabled: Screenshot_20210414_080246 Extension disabled: Screenshot_20210414_080311

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:

Screenshot_20210414_080824

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

posted by SimonBrandner almost 4 years ago

Hmm looking at the code I don't think that should happen.

Are you able to copy the DOM here?

posted by yakov116 almost 4 years ago

Here you go

posted by SimonBrandner almost 4 years ago

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?

posted by yakov116 almost 4 years ago

Here. Thanks for looking into this, btw!

posted by SimonBrandner almost 4 years ago
posted by yakov116 almost 4 years ago

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">

posted by fregante almost 4 years ago

I don’t think it's enterprise only. Triage is a type of collaborator

posted by fregante almost 4 years ago

I would try to fix it. Do you have a repo you can add me on to debug?

posted by yakov116 almost 4 years ago

@fregante can you reenable and disable via hotfix?

posted by yakov116 almost 4 years ago

I'm not sure I understand

posted by fregante almost 4 years ago

We commented out the line

posted by yakov116 almost 4 years ago

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

posted by fregante almost 4 years ago

Correct but hotfixing it will allow users to know it was disabled and why.

posted by yakov116 almost 4 years ago

Thank you!

posted by SimonBrandner almost 4 years ago

Fund this Issue

$0.00
Funded

Pull requests