sindresorhus/refined-github

`global-conversation-list-filters` creates faulty links #3731

KSH-code posted onGitHub

<img width="1679" alt="스크린샷 2020-11-18 오후 1 43 31" src="https://user-images.githubusercontent.com/29705162/99484581-134b2800-29a4-11eb-893f-5da11de31320.png"> In my review requests tab of PRs menu, whenever to click commented tab, the error occurs.

How to fix that? or already has it fixed?


Using extension's version is 20.11.12 and my chrome's version is 86.0.4240.111.


It will be fixed in the next release (tomorrow)

posted by yakov116 over 4 years ago

Thank you for help. close it, because my problem's been solved.

posted by KSH-code over 4 years ago

@yakov116 The bug has not been solved until now in my Chrome browser. Is this fixed?

the bug cannot be shown on console but commented list is not updated.

the error would be shown in situation.

  1. Click Pull Requests tab.
  2. Click Commented tab.
  3. Click Review Requests tab
  4. Click Commented tab.

in 4th situation, PRs list is not updated.

posted by KSH-code almost 4 years ago

Not happening by me

posted by yakov116 almost 4 years ago

Can you please paste the error here?

posted by yakov116 almost 4 years ago

Could there be an issue with a dashed username?

posted by fregante almost 4 years ago

Can you paste the URLs of the steps 2 and 4? We will know exactly what's happening

posted by fregante almost 4 years ago
  1. https://github.com/pulls/review-requested
  2. https://github.com/pulls/review-requested?q=is%3Apr+is%3Aopen+archived%3Afalse+commenter%3AKSH-code

when 2, Review Requests tab is still active. @fregante @yakov116

There is no error in console.

I have just found the bug with Yours tab as well.

posted by KSH-code almost 4 years ago

This piece of code is not working:

https://github.com/sindresorhus/refined-github/blob/ae020ab5d6c851045b39cd794e384e7aa4c91f1e/source/features/global-conversation-list-filters.tsx#L35-L38

Or rather, the SearchQuery is probably unable to correctly remove the commenter: query due to the dash in the username

posted by fregante almost 4 years ago

@fregante Hey, I'm back here issue. I was wondering this issue can be fixed soon. If no and I have time to fix this, I will fix on my own. but I'm newbie about TS. so to fix it will spend lots of time. 😂

posted by KSH-code almost 4 years ago

The problem doesn't seem to be what I thought. This SearchQuery test passes correctly:

test('.remove with dash', t => {
    // Native "Review requested" query after user visits the `Commented` tab
    const query = new SearchQuery({q: 'is:pr is:open archived:false commenter:KSH-code sort:updated-desc review-requested:KSH-code'});

    // Replicates line 37 shown above
    query.remove('commenter:KSH-code');

    t.is(query.get(), 'is:pr is:open archived:false sort:updated-desc review-requested:KSH-code');
});
posted by fregante almost 4 years ago

@fregante Hey, It's been a while. The problem has been solved by update via Chrome developer mode. It occured when pathname ends with /pulls/review-requested but now, only pathname represents /pulls

posted by KSH-code almost 4 years ago

Oooops, I have just found the bug. The reason why the bug occurs is because of sort option. I'll fix soon.

posted by KSH-code almost 4 years ago

Thanks ! :)

posted by KimGenius almost 4 years ago

Fund this Issue

$0.00
Funded

Pull requests