Merge lp://staging/~facundo/ubuntu-rest-scopes/bye-bye-7digital into lp://staging/ubuntu-rest-scopes

Proposed by Facundo Batista
Status: Merged
Approved by: Facundo Batista
Approved revision: 544
Merged at revision: 542
Proposed branch: lp://staging/~facundo/ubuntu-rest-scopes/bye-bye-7digital
Merge into: lp://staging/ubuntu-rest-scopes
Diff against target: 126 lines (+32/-6)
5 files modified
configs/dev-secrets.yaml (+1/-1)
src/scopes/__init__.py (+7/-0)
src/scopes/sevendigital.py (+6/-2)
src/scopes/tests/test_base.py (+11/-1)
src/scopes/tests/test_sevendigital.py (+7/-2)
To merge this branch: bzr merge lp://staging/~facundo/ubuntu-rest-scopes/bye-bye-7digital
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
Guillermo Gonzalez Approve
Review via email: mp+327293@code.staging.launchpad.net

Commit message

Support a misconfiguration error, returning empty, and made 7digital to get into it.

Description of the change

Support a misconfiguration error, returning empty, and made 7digital to get into it.

To post a comment you must log in.
543. By Facundo Batista

Removed debug print line.

544. By Facundo Batista

And removed other annotations.

Revision history for this message
Guillermo Gonzalez (verterok) wrote :

+1

review: Approve
Revision history for this message
Daniel Manrique (roadmr) wrote :

I like it, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches