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

removes ToU from the main text while keeping it in the reserved extension points table #1294

Closed
wants to merge 1 commit into from

Conversation

Sakurann
Copy link
Contributor

@Sakurann Sakurann commented Sep 26, 2023

addresses issue #1010 - there does not seem to be sufficient interest to better define Terms of Use property in the main text.
but keeping it in the reserved extension points table.


Preview | Diff

@David-Chadwick
Copy link
Contributor

I suggest that this PR is closed due to #1295

@decentralgabe decentralgabe added the DO NOT MERGE PR contains something that should not be merged. label Oct 9, 2023
Copy link
Member

@msporny msporny left a comment

Choose a reason for hiding this comment

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

Suggest that we close this PR since PR #1295 has overtaken it.

@brentzundel brentzundel added the pending close Close if no objection within 7 days label Oct 17, 2023
@iherman
Copy link
Member

iherman commented Oct 17, 2023

The issue was discussed in a meeting on 2023-10-17

  • no resolutions were taken
View the transcript

2.5. removes ToU from the main text while keeping it in the reserved extension points table (pr vc-data-model#1294)

See github pull request vc-data-model#1294.

Brent Zundel: The plan as we discussed last time, is to close 1294 assuming we get consensus and can merge 1295.
… I'm going to mark it as pending close.
… The discussion continues in 1295.

Manu Sporny: +1 to that path forward.

@Sakurann Sakurann closed this Oct 26, 2023
@msporny msporny deleted the sakurann-remove-ToU branch November 11, 2023 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE PR contains something that should not be merged. pending close Close if no objection within 7 days
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants