← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~abentley/launchpad/transitive-confidential into lp:launchpad

 

Review: Approve code

Looks good to land.

I just have concern: If we assume that somebody creates a public project, adds public bugs, branches, specifications, links packages and enables translations.

If this person decides to make product proprietary, the user experience is terrible:

The first error is "sorry, no public specs allowed", so the user changes the specifications.

The next error then is "sorry, no public bugs allowed" and so on. It would be better to collect all "show stoppers" and tell the user immediately that they have to fix issues with bugs, branches, specs, translations and packages.
-- 
https://code.launchpad.net/~abentley/launchpad/transitive-confidential/+merge/135926
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.


References