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

feat: prism-agent-overview draft 1 #14

Merged
merged 1 commit into from
Mar 2, 2023
Merged

Conversation

lohanspies
Copy link
Contributor

@lohanspies lohanspies commented Feb 16, 2023

Fixes ATL-3535
@lohanspies have the engineering provided what you needed for the list of Aries RFCs and DIDComm protocols supported in Prism Agent as I see that both Anton and Yurii have reviewed?

@lohanspies
Copy link
Contributor Author

@davidpoltorak-io this PR requires engineering to complete the list of Aries RFCs and DIDComm protocols supported in Prism Agent. Can you please ask them to get it completed?

@lohanspies
Copy link
Contributor Author

@yshyn-iohk, what is more important is to review the bottom section where we list the various Aries RFCs supported etc. Can you please go through the list of supported RFCs and add them to the overview document?

Copy link
Member

@yshyn-iohk yshyn-iohk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good enough for the first version.
I propose to close solve the conversations and remove the things that are not ready at all (AIP 1.0 AIP 3.0).
If we plan to do something, it's better to keep it in the roadmap document.

@yshyn-iohk
Copy link
Member

@lohanspies, this was discussed in Slack here: https://input-output-rnd.slack.com/archives/C012T7UP886/p1676937571790169?thread_ts=1676903933.903319&cid=C012T7UP886

I propose to merge the current PR and add the concrete protocols in the next PR.

@lohanspies
Copy link
Contributor Author

@yshyn-iohk I can take a stab at updating what I can from the Slack list provided.
Then we can review and update in another PR?
Thoughts?

@lohanspies
Copy link
Contributor Author

@lohanspies, this was discussed in Slack here: https://input-output-rnd.slack.com/archives/C012T7UP886/p1676937571790169?thread_ts=1676903933.903319&cid=C012T7UP886

I propose to merge the current PR and add the concrete protocols in the next PR.

@yshyn-iohk create an initial version of supported specs and pushed as part of this PR. We can review and update later.

Copy link
Contributor

@antonbaliasnikov antonbaliasnikov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I propose we merge this. And incrementally enhance later.

@yshyn-iohk yshyn-iohk force-pushed the feature/prism-agent-overview branch from 0e8b756 to cc8f4d6 Compare March 2, 2023 09:48
@yshyn-iohk yshyn-iohk merged commit 59a20d2 into main Mar 2, 2023
@yshyn-iohk yshyn-iohk deleted the feature/prism-agent-overview branch March 2, 2023 09:54
atala-dev added a commit that referenced this pull request Mar 3, 2023
…[1.1.0](v1.0.0...v1.1.0) (2023-03-03)

### Bug Fixes

* update prism-did method URL ([#15](#15)) ([af5bd09](af5bd09))

### Features

* add glossary ([#17](#17)) ([8e8cdd4](8e8cdd4))
* implemented release and deploy on main commit ([#20](#20)) ([cf0e2bc](cf0e2bc))
* prism mediator overview ([#13](#13)) ([3a7abe2](3a7abe2))
* prism-agent-overview draft 1 ([#14](#14)) ([59a20d2](59a20d2))
* remove unused arch chapters and add running locally ([#19](#19)) ([bf4a27b](bf4a27b))
* update submodules documentation to latest ([1b322e1](1b322e1))
* update submodules documentation to latest ([83109e9](83109e9))
* update submodules documentation to latest ([1ff29d7](1ff29d7))
* update verifiable-data.md ([#16](#16)) ([0471b35](0471b35))
* updated copy ([#21](#21)) ([4eb6261](4eb6261))
* Updates to prism-node, bbs and verfiable data concept ([#12](#12)) ([87a5fde](87a5fde))
* use tutorials from building blocks repo ([e8948e1](e8948e1))
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.

4 participants