Bug #5493
closedRhodeCode removes pull requests when the origin fork is deleted.
0%
Description
This looks like it is built as a feature, not sure if it was by design or on accident. However, it is in-fact a huge issue with the workflow of RhodeCode.
If I fork a repository, make some changes then open a pull request and ultimately get that pull request to be accepted and merged into the upstream repository, I would expect that no matter what happens to my fork, the pull request will continue to be accessible for historical reference of that merge.
However, if on completion of the merge I decide to delete my fork, all data pertaining to that fork is also removed from the database. This includes the entries in the pull_requests and changeset_comments tables. It effectively blows away all historical data for that merge. That is terrible! If anything the entries in the database should be marked as "deleted" rather than just a straight up row delete. I was pretty shocked to see this.
Updated by Marcin Kuzminski [CTO] over 6 years ago
- Target version set to v4.14
Yes, the Deletion purges the repo content, and all attached repositories.
We know this might be not ideal, and as you pointed out keeping history is important. That is why we needed first to "save" the PR results even if the source/targets are not availably anymore, this was introduced in 4.13 as diff-caching. We'll introduce now an option to archive a repository, and not DELETE it, so it would keep all references. Also we'll make DELETE explicitly state what is going to be removed/detached.
We plan this for 4.14 in next 4-8 weeks.
Updated by Redmine Integration about 6 years ago
- Status changed from New to Closed
Commit e665305df644
by Marcin Kuzminski marcin@rhodecode.com on default
branch changed this issue.
https://code.rhodecode.com/rhodecode-enterprise-ce/changeset/e665305df644eeb600c2743eecd5928dc70a7808
Commit bdd9dc16c9e6
by Marcin Kuzminski marcin@rhodecode.com on default
branch changed this issue.
https://code.rhodecode.com/rhodecode-enterprise-ce/changeset/bdd9dc16c9e6c480d2daa907c00b1e5711fe1b6a