Merge ~ines-almeida/launchpad:db-add-social-accounts-table into launchpad:db-devel
Proposed by
Ines Almeida
| Status: | Merged |
|---|---|
| Approved by: | Guruprasad |
| Approved revision: | e2c970e25ae82b725edf5d9ff55d35446729eb48 |
| Merge reported by: | Otto Co-Pilot |
| Merged at revision: | not available |
| Proposed branch: | ~ines-almeida/launchpad:db-add-social-accounts-table |
| Merge into: | launchpad:db-devel |
| Diff against target: |
12 lines (+1/-0) 1 file modified
database/schema/security.cfg (+1/-0) |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Guruprasad | Approve | ||
| William Grant | db | Approve | |
|
Review via email:
|
|||
Commit message
db: add DB table permissions to person-merge-job profile
Description of the change
A few unit tests are still failing due to table permissions for the new socialaccounts table.
Looking through the failing tests, apart from a few intermittent failures, they seem to come mostly from the person merge job. I tested a handful of the failing tests locally
To post a comment you must log in.
Revision history for this message
| William Grant (wgrant) | # |
review:
Approve
(db)
Revision history for this message
| Otto Co-Pilot (otto-copilot) wrote | # |
Revision history for this message
| Guruprasad (lgp171188) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Voting criteria not met
https:/