Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add create-proposal endpoint to the present-proof API namespace #288

Merged
merged 1 commit into from
Nov 30, 2019

Conversation

esune
Copy link
Member

@esune esune commented Nov 29, 2019

Adding a new API endpoint /present-proof/create-request to support creating a Proof request without needing an active connection with an agent, as previously supported by the deprecated endpoint /presentation_exchange/create_request.

tests have been added following the ones that were already available for presentation_exchange_send_free_request and removing the test cases that are not required due to the missing connection-id requirement.

Please note my formatter has made changes to the files I edited (currently using black version 19.3b0): let me know if this is expected, or what I should be doing to have a formatter behaviour that is consistent with the rest of the community.

Tagging @andrewwhitehead for review.

@esune
Copy link
Member Author

esune commented Nov 29, 2019

Changes in this PR depend on this aca-py pull request.

@codecov-io
Copy link

codecov-io commented Nov 29, 2019

Codecov Report

Merging #288 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #288      +/-   ##
==========================================
+ Coverage   81.61%   81.64%   +0.02%     
==========================================
  Files         232      232              
  Lines       10770    10785      +15     
==========================================
+ Hits         8790     8805      +15     
  Misses       1980     1980

@andrewwhitehead andrewwhitehead merged commit e1f6f3c into openwallet-foundation:master Nov 30, 2019
esune added a commit to esune/acapy that referenced this pull request Dec 10, 2019
@esune esune mentioned this pull request Dec 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants