Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix new-repo-disable-projects-and-wiki feature - #3794

Merged
yakov116 merged 2 commits into
refined-github:masterfrom
cheap-glitch:fix-new-repo-disable-projects-and-wiki
Dec 9, 2020
Merged

Fix new-repo-disable-projects-and-wiki feature#3794
yakov116 merged 2 commits into
refined-github:masterfrom
cheap-glitch:fix-new-repo-disable-projects-and-wiki

Conversation

@cheap-glitch

Copy link
Copy Markdown
Contributor

Thanks for contributing! 馃崉

  1. LINKED ISSUES: none

  2. TEST URLS: https://github.com/new

The new-repo-disable-projects-and-wiki feature was the only one passing an empty query string to api.v3(), which caused a trailing slash to appear at the end of the URL and resulted in a 404 error.

@yakov116

yakov116 commented Dec 7, 2020

Copy link
Copy Markdown
Member

Just pass a slash instead of an empty string in the feature

@cheap-glitch

Copy link
Copy Markdown
Contributor Author

Just pass a slash instead of an empty string in the feature

This won't work. We want 'repos/' + getRepo()!.nameWithOwner, not an empty URL.

@yakov116

yakov116 commented Dec 7, 2020

Copy link
Copy Markdown
Member

Oh your right misunderstood the issue

Comment thread source/github-helpers/api.ts Outdated
@yakov116 yakov116 added the bug label Dec 8, 2020
@yakov116 yakov116 changed the title Fix new-repo-disable-projects-and-wiki Fix new-repo-disable-projects-and-wiki feature Dec 9, 2020
@yakov116
yakov116 merged commit 4d95862 into refined-github:master Dec 9, 2020
@yakov116

yakov116 commented Dec 9, 2020

Copy link
Copy Markdown
Member

Thanks @cheap-glitch

@cheap-glitch
cheap-glitch deleted the fix-new-repo-disable-projects-and-wiki branch December 9, 2020 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants