1 A middle page may be necessary like GH did? Looks like GH will not display a fork form page and will fork at the backend.

2 The branches may be fixed asAll branches to prevent to choose a wrong branch

3 Is that really necessary to allow user to fork to an organization when forking for the edit?

4 The edit button on README file of repository home page could be displayed.

5 Too many steps and it will make some beginner confusing.

I rewrote the whole thing, which should hopefully address all the comments.

I think the permissions checking is rather confusing now. Previouslyctx.Repo.CanWriteToBranch,ctx.Repo.CanEnableEditor andctx.Repo.MustBeEditable were easy to understand, but now that two repositories are involved it's no so simple anymore. Maybe I should add actx.Repo.UserFork into the context, so that these functions can use it? Suggestions welcome.

Fork repository was created after clicking thefork the repository button. But it redirects to 404 . And when I re edit the file,

The "fork" logic has too much legacy patched code, it seems that it is not good to introduce more patches, instead maybe it needs refactoring ....

And I think it needs tests to make sure the logic is overall right.

Do we have enough time to take it into 1.24? (TBH I guess no)

Fork repository was created after clicking thefork the repository button. But it redirects to 404 .

Fixed now, with test.

The "fork" logic has too much legacy patched code, it seems that it is not good to introduce more patches, instead maybe it needs refactoring ....

Ok, I'm not familiar enough with the overall code to see what is legacy here. Some hints or help would be appreciated.

Do we have enough time to take it into 1.24? (TBH I guess no)

I'm not in a rush to put it in 1.24.

The edit buttons might be disabled for anonymous users. I’m not sure if they should be, but GitHub does it that way. Otherwise LGTM

• TheAdd File button is not visible on the base repo, only when you have permission to edit it directly.
• I'm unsure ifCherry Pick is supposed to work. I didn't implement support for doing that in this work flow. This new code doesn't seem to prohibit it in the router, but also doesn't expose it in the UI.
• TheYou can not edit this repository directly. The changes will be written to your fork text is now at the bottom of the page. Was than an intentional change? Personally I think it's useful information to see before you start editing. GitHub also shows it at the top though the overall UI is different.
• When specifying e.g.main in the new branch name, it succeeds even though that branch already exists.
• When I disable the "Code" unit on my fork, it lets me edit the file, but when clicking "Propose file changes" I getNetwork error SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data.
• My version added tests for some features that were not previously tested. It might be useful to salvage some of those even if it's just the non-fork cases.

I don't really have anything to mention regarding the code. It's much simpler now and the hard work seems to have been done in the refactoring. Many thanks!

TheAdd File button is not visible on the base repo, only when you have permission to edit it directly.

Fixed and added a test

I'm unsure ifCherry Pick is supposed to work. I didn't implement support for doing that in this work flow. This new code doesn't seem to prohibit it in the router, but also doesn't expose it in the UI.

They share the same backend logic (the same as "Apply Patch"), so I think it's fine to leave it as-is.

We can expose it in the future if anyone really needs it.

TheYou can not edit this repository directly. The changes will be written to your fork text is now at the bottom of the page. Was than an intentional change? Personally I think it's useful information to see before you start editing. GitHub also shows it at the top though the overall UI is different.

Moved to the top (see the screenshot)

When specifying e.g.main in the new branch name, it succeeds even though that branch already exists.

Fixed and added some tests.

When I disable the "Code" unit on my fork, it lets me edit the file, but when clicking "Propose file changes" I getNetwork error SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data.

Improved the frontend error handling (see the screenshot)

My version added tests for some features that were not previously tested. It might be useful to salvage some of those even if it's just the non-fork cases.

I will try to pick some tests later. And could you help to list the tests you think are important?

Screenshots (slightly outdated, the new commits moved the prompt to before the editor header line)

My version added tests for some features that were not previously tested. It might be useful to salvage some of those even if it's just the non-fork cases.

I will try to pick some tests later. And could you help to list the tests you think is important?

Made some refactorings toeditor_test.go and picked most of your new tests. Are there somethings more we need to take a look at?

Looks great now, no more comments from my side.

Let's merge and try (and fix regressions 😁 )