Skip to content
This repository has been archived by the owner on Dec 11, 2024. It is now read-only.

Add description for how to use X.509 certificates to README #62

Merged
merged 2 commits into from
Oct 25, 2022

Conversation

gpeacock
Copy link
Contributor

@gpeacock gpeacock commented Sep 8, 2022

Changes in this pull request

Restores signature setup information and adds other missing info

Checklist

  • This PR represents a single feature, fix, or change.
  • All applicable changes have been documented.
  • Any TO DO items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.

@scouten-adobe scouten-adobe changed the title Readme updates Add description for how to use X.509 certificates to README Sep 8, 2022
README.md Outdated Show resolved Hide resolved
@crandmck
Copy link
Collaborator

crandmck commented Sep 29, 2022

@gpeacock I reviewed the changes in the PR and resolved the merge conflicts with the current README. I also incorporated the comment from @mauricefisher64. I made a few other small edits for clarity and conciseness.

After your final review, this should be ready to merge.

Copy link
Contributor

@mauricefisher64 mauricefisher64 left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor Author

@gpeacock gpeacock left a comment

Choose a reason for hiding this comment

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

Looks good except for we need to explain that the tool now retrieves cloud manifests.

README.md Show resolved Hide resolved
@gpeacock gpeacock merged commit fac6704 into main Oct 25, 2022
@gpeacock gpeacock deleted the gpeacock/readme_update2 branch October 25, 2022 16:14
scouten-adobe pushed a commit to contentauth/c2pa-rs that referenced this pull request Dec 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants