CodeTriage icon indicating copy to clipboard operation
CodeTriage copied to clipboard

Merge Active Admin pages

Open seanlinsley opened this issue 10 years ago • 6 comments

Active Admin used to exist at http://github.com/gregbell/active_admin, but has since moved to http://github.com/activeadmin/activeadmin.

There are currently two separate pages on CodeTriage, that have different subscribers:

  • 65 subscribers, 221 open issues: http://www.codetriage.com/gregbell/active_admin
  • 15 subscribers, 47 open issues: http://www.codetriage.com/activeadmin/activeadmin

Ideally these should be merged together.

Why does the newer page only show 47 open issues even though currently there are 183 on Github? Have the people subscribed to the old repo name been getting tickets to triage?

seanlinsley avatar Oct 13 '15 16:10 seanlinsley

We need to merge these. There is no user facing feature to do so, but we should add one.

schneems avatar Oct 13 '15 16:10 schneems

Is there anything I can do to help?

seanlinsley avatar Oct 22 '15 21:10 seanlinsley

The code logic to merge is pretty easy

old_repo = Repo.where(full_name: "gregbell/active_admin")
new_repo = Repo.where(full_name: "activeadmin/activeadmin")

old_repo.users.find_each do |user|
  new_repo << user
end

The interaction logic is harder. I would say the original person who added the repos on codetriage can merge and the owners of the original repo. I'm not sure if we should delete the old repo, we should probably add an "archived" state instead to soft delete in case of accidents.

schneems avatar Nov 04 '15 15:11 schneems

Well the best (but more complex) solution would be to redirect the old URL to the new URL, just like Github itself does.

seanlinsley avatar Nov 05 '15 17:11 seanlinsley

Add a "redirect" field to the Repo model. When it is present send to that repo (provided that repo exists so we don't open ourselves up to an open redirect attack).

schneems avatar Nov 06 '15 18:11 schneems

@SagareGanesh If you are interested, please take a look at this issue 😁

prathamesh-sonpatki avatar Dec 18 '16 07:12 prathamesh-sonpatki