sindresorhus/refined-github

Restore `quick-review-button` feature #2700

avaly posted onGitHub

I can not approve/request changes/comment on any pull request.

github-refined-bug-review

The behavior works if I disable the add-on.

Browser: Firefox 72.0.1 Add-on version: 20.1.14


Screen Shot 2020-01-14 at 11 23 31

Browser: Firefox Developer Edition 73.0b4 (64-bit) Add-on version: 20.1.14

posted by mauriciord about 5 years ago

Same here, looks like the quick-review-button feature is having issues. I disabled it and the "old" buttons are working.

Browser: Firefox Developer Edition 73.0b4 (64-bit) Add-on version: 20.1.14

posted by dsiebel about 5 years ago

I just pushed a hotfix. Chrome isn't affected because 20.1.14 was rejected.

Try updating soon, you should see 20.1.14.1820

posted by fregante about 5 years ago

Thanks for the quick reaction! I am on 20.1.14.1820 now, but I can not re-enable the feature.
It says its "Disabled because of #2700" and I can not tick the box:

<img width="644" alt="Screen Shot 2020-01-15 at 11 52 55" src="https://user-images.githubusercontent.com/145283/72428274-99e41700-378d-11ea-946d-35903adcf5e6.png">

(or was the hotfix to disable it?)

posted by dsiebel about 5 years ago

@dsiebel hot fix was to disable

posted by yakov116 about 5 years ago

I think this bug is due to https://github.com/sindresorhus/refined-github/pull/2639

I haven’t had time to investigate so I just disabled the feature since it broke GitHub.

posted by fregante about 5 years ago

It was actually https://github.com/sindresorhus/refined-github/commit/4c502f50b45d339f9eb6fb3964b4eabc7ba7a36f

It forced me to add a type=button attribute on buttons, which removes the default submit behavior; in 2 cases, the button actually had to submit.

posted by fregante about 5 years ago

It was actually 4c502f5

It forced me to add a type=button attribute on buttons, which removes the default submit behavior; in 2 cases, the button actually had to submit.

@fregante Did you look into configuration the button-has-type eslint plugin? https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/button-has-type.md

posted by mobilutz about 5 years ago

Fund this Issue

$0.00
Funded

Pull requests