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.

Sign up for GitHub

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

Jump to bottom

Authorize RecordsWrites with protocol roles #524

Merged
merged 8 commits into from
Oct 4, 2023
Merged

Authorize RecordsWrites with protocol roles #524

merged 8 commits into from
Oct 4, 2023

Conversation

diehuxx
Copy link

@diehuxx diehuxx commented Sep 29, 2023

No description provided.

@codesandbox
Copy link

codesandbox bot commented Sep 29, 2023

Review or Edit in CodeSandbox

Open the branch in Web Editor • VS Code • Insiders

Open Preview

thehenrytsai
thehenrytsai reviewed Oct 3, 2023
View reviewed changes
README.md Show resolved Hide resolved
@diehuxx diehuxx force-pushed the diehuxx/role-write branch from de7cd64 to 206b4c4 Compare October 3, 2023 23:26
thehenrytsai
thehenrytsai previously approved these changes Oct 4, 2023
View reviewed changes
Merge branch 'main' into diehuxx/role-write
35fe439 
* main:
  #526 - Fix typos in the README.md file (#527)
  remove unnecessary datastream from being passed for ProtocolsConfigure (#528)
Diane Huxley added 2 commits October 4, 2023 10:41
Slight test coverage boost
ad4b547
Merge branch 'main' into diehuxx/role-write
1dae73f 
* main:
  #439 - Appended 'Signature' to` author` and `owner` under `authorization` (#536)
  #439 - Added `RecordsWrite` support for retainer signature (#531)
@diehuxx diehuxx merged commit 17199ee into main Oct 4, 2023
diehuxx pushed a commit to adarsh-a-tw/dwn-sdk-js that referenced this pull request Oct 4, 2023
Merge branch 'main' into 530-normalize-schema-and-protocol
6b161b7 
* main:
  Authorize RecordsWrites with protocol roles (decentralized-identity#524)
@thehenrytsai thehenrytsai deleted the diehuxx/role-write branch April 8, 2024 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thehenrytsai thehenrytsai thehenrytsai approved these changes

@mistermoe mistermoe Awaiting requested review from mistermoe mistermoe is a code owner

@csuwildcat csuwildcat Awaiting requested review from csuwildcat csuwildcat is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@diehuxx @thehenrytsai