launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #14512
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