Code review comment for lp:~edwin-grubbs/launchpad/bug-525956-unlink-button

Revision history for this message
Curtis Hovey (sinzui) wrote :

Hi Edwin.

This this was a nice feature to use on the sp page. I think we should consider using it on the dsp page: https://launchpad.dev/ubuntu/+source/evolution. We have a post back that some people have expressed concern about how easy it is to make a mistake. I would not say this is change we must do because the post back always returns the user to the dsp page--using the sp as the next_url would be confusing.

I think we should remove the edit/remove links from the +packaging listing. There is not enough information on this page to make that decision.

review: Needs Information (ui)

« Back to merge proposal