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: update extension packages docs #129

Merged

Conversation

TimoGlastra
Copy link
Contributor

Simplify structure, update to work with new versions of packages, and move some docs over from the readme of the packages

Copy link
Contributor

@berendsliedrecht berendsliedrecht left a comment

Choose a reason for hiding this comment

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

Seems like it is near identical to the README. I think it is fine to have it like the as you will have one place for all the docs, but keeping it in sync might become a bit annoying. Would be nice if we can have urls for the markdown we would like to render.

@TimoGlastra
Copy link
Contributor Author

Seems like it is near identical to the README. I think it is fine to have it like the as you will have one place for all the docs, but keeping it in sync might become a bit annoying. Would be nice if we can have urls for the markdown we would like to render.

Yes, my idea was to remove it from the README and always point to the docs. that way it's in one place and the README is really simple

@TimoGlastra TimoGlastra enabled auto-merge (squash) August 17, 2023 13:19
@TimoGlastra TimoGlastra merged commit 9539eb1 into openwallet-foundation:main Aug 17, 2023
2 checks passed
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.

2 participants