gitea
gitea copied to clipboard
go-gitea
add org level milestones
https://github.com/go-gitea/gitea/issues/20203
https://github.com/go-gitea/gitea/assets/20443971/34f14dd5-53ce-4ecd-bbc7-e5b372f8e463
PR sponsored by Obmondo.com
This PR is coming from an org, so maintainers have no push permission to the source branch. Can this be allowed somehow? Would make things easier.
This PR is coming from an org, so maintainers have no push permission to the source branch. Can this be allowed somehow? Would make things easier.
i have no idea, let me ask someone
This PR is coming from an org, so maintainers have no push permission to the source branch. Can this be allowed somehow? Would make things easier.
So you want to push to "source branch" in source repo (the fork) ? that sounds very odd.. that would not work even if I forked it to my personal repo ? that sounds weird.. :)
If you know of any org settings for the org - I can change them though as I am org owner though.
@iminfinity will fix merge conflict and other things, so its good to merge right?
in this case - @iminfinity will have to update our fork of latest main and then re-apply/adjust his commits to work (and test ofcourse) - and then push the updated change so it matches
Maybe it'd be easier if you re-raise the PR from a user account so that we can push fixes onto this branch. I don't know why GitHub has this silly restriction to not allow collaboration on org branches.
I could try to explain more: we encourage contributors to enable "Allow maintainers to edit" for their PRs, then maintainers could help to improve the code together and update&merge.
|
|
|
However, GitHub has a restriction: if a PR is from an org, then no maintainer could edit that PR. That's why the PRs from an org are difficult to review&improve.
If a PR is from a user's account and allows maintainers to edit, then some maintainers who have interests and experiences could work together to try to complete it.
And one more thing, there are still many problems, for example: the db migration should use XORM sync, many CSS styles are no long available ("gt-mr-2" / "mr-2"), coding-style (use template {{}} instead of print), duplicate code (some code blocks are copied&pasted?), strange key/variable (orgPath), etc. And it's better to have some tests to cover the new logic.
Haven't really taken a look carefully, but I guess it's far from mature.
Stale for long time, would you like to correct the problems? Or close it?
Thank you for the PR 🙏 Feel free to reopen if there would be new progress.
sorry, ive been busy, have not had any time for this, there are some bugs on this pr and i have fixed it in our fork, i will clean this up when i have the time and reopen it, cuz we at obmondo have been using global milestone for more than a year now and would like everyone in the community to be able to use it as well
Hi @iminfinity , our org would also greatly benefit from this feature! Hopefully you have time to finish this soon.
