Don't require approval for VCS imports
Bug #418932 reported by
Jonathan Lange
This bug affects 3 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Medium
|
Jelmer Vernooij | ||
Bug Description
Currently, VCS imports need to be approved by a select team of experts. They should not require this step.
Related branches
lp:~jelmer/launchpad/no-code-import-approval
- Benji York (community): Approve (code)
-
Diff: 581 lines (+81/-125)13 files modifiedlib/lp/code/doc/codeimport-event.txt (+5/-5)
lib/lp/code/doc/codeimport.txt (+1/-5)
lib/lp/code/model/codeimport.py (+4/-8)
lib/lp/code/model/tests/test_branch.py (+0/-2)
lib/lp/code/model/tests/test_codeimport.py (+12/-40)
lib/lp/code/model/tests/test_codeimportjob.py (+16/-11)
lib/lp/code/stories/codeimport/xx-admin-codeimport.txt (+3/-20)
lib/lp/code/stories/codeimport/xx-create-codeimport.txt (+9/-4)
lib/lp/code/stories/codeimport/xx-edit-codeimport.txt (+3/-1)
lib/lp/code/stories/webservice/xx-code-import.txt (+2/-2)
lib/lp/testing/factory.py (+11/-13)
lib/lp/testing/tests/test_factory.py (+4/-3)
utilities/sourcedeps.cache (+11/-11)
Revision history for this message
| Jonathan Lange (jml) wrote | #1 |
Revision history for this message
| Karl Fogel (kfogel) wrote | #2 |
Revision history for this message
| Jelmer Vernooij (jelmer) wrote | #3 |
| Changed in launchpad: | |
| status: | Triaged → In Progress |
| assignee: | nobody → Jelmer Vernooij (jelmer) |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #4 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Jelmer Vernooij (jelmer) wrote | #5 |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
I'm happy to mentor someone through this change.