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

Add troubleshooting document, include initial examples - ledger connection, out-of-sync RevReg #1818

Merged
merged 4 commits into from
Jun 21, 2022

Conversation

swcurran
Copy link
Contributor

Signed-off-by: Stephen Curran [email protected]

@swcurran
Copy link
Contributor Author

@WadeBarnes @ianco -- please take a look. In a comment here, feel free to suggest other troubleshooting problems that we should include in this document.

Is the document a good idea? Is the content sufficient? I couldn't think of where else to document the RevReg fixing stuff.

I believe we need an issue (and a PR) that is intended to prevent the "out of sync" scenario - correct?

@codecov-commenter
Copy link

codecov-commenter commented Jun 16, 2022

Codecov Report

Merging #1818 (31ac61e) into main (5ac6086) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1818   +/-   ##
=======================================
  Coverage   93.71%   93.71%           
=======================================
  Files         534      534           
  Lines       33890    33890           
=======================================
  Hits        31761    31761           
  Misses       2129     2129           

Copy link
Contributor

@WadeBarnes WadeBarnes left a comment

Choose a reason for hiding this comment

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

Troubleshooting.md Outdated Show resolved Hide resolved
Troubleshooting.md Outdated Show resolved Hide resolved
Troubleshooting.md Outdated Show resolved Hide resolved
Troubleshooting.md Outdated Show resolved Hide resolved
@swcurran
Copy link
Contributor Author

@WadeBarnes -- please take another look.

WadeBarnes
WadeBarnes previously approved these changes Jun 21, 2022
@swcurran swcurran merged commit 9364295 into openwallet-foundation:main Jun 21, 2022
@swcurran swcurran deleted the troubleshooting branch June 21, 2022 22:26
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.

3 participants