Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new-repo-disable-projects-and-wikis feature #3539

Merged
merged 14 commits into from
Sep 14, 2020

Conversation

yakov116
Copy link
Member

@yakov116 yakov116 commented Sep 10, 2020

source/features/new-repo-disable-projects-and-wikis.tsx Outdated Show resolved Hide resolved
source/features/new-repo-disable-projects-and-wikis.tsx Outdated Show resolved Hide resolved
source/features/new-repo-disable-projects-and-wikis.tsx Outdated Show resolved Hide resolved
source/features/new-repo-disable-projects-and-wikis.tsx Outdated Show resolved Hide resolved
source/features/new-repo-disable-projects-and-wikis.tsx Outdated Show resolved Hide resolved
source/features/new-repo-disable-projects-and-wikis.tsx Outdated Show resolved Hide resolved
readme.md Outdated Show resolved Hide resolved
fregante and others added 2 commits September 10, 2020 18:09
+ don't use `name` in forms we don't own; its value will be sent to GitHub
@yakov116
Copy link
Member Author

Not sure why typescript is failing

@fregante
Copy link
Member

Because it returns a Node, which doesn’t have .remove; it should be an Element. Also make a select-like helper function because 2 consecutive 182-characters lines are obviously not ok.

@yakov116

This comment has been minimized.

Co-authored-by: Federico <me@fregante.com>
@yakov116
Copy link
Member Author

@fregante just saying thank you for all your detailed reviews! 🚀

@fregante fregante merged commit 11020f9 into refined-github:master Sep 14, 2020
@yakov116 yakov116 deleted the newrepo branch September 14, 2020 20:06
@fregante
Copy link
Member

I think this stopped working and I'm not seeing any related console errors

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Automatically disable wiki and project features
2 participants