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

Fix logic of checking whether non-participants can view order details #245

Merged
merged 3 commits into from
Sep 15, 2022

Conversation

redphix
Copy link
Contributor

@redphix redphix commented Sep 14, 2022

Not sure if this is a typo or am I missing something(?)

@redphix redphix marked this pull request as ready for review September 14, 2022 18:00
@Reckless-Satoshi
Copy link
Collaborator

It's a typo/bug, nice find.
Added a PR with one more commit to your branch to polish this patch (any known workarounds so I can add commits directly to a PR?)

@redphix
Copy link
Contributor Author

redphix commented Sep 15, 2022

I have enabled this feature, but not sure how it works though. My guess is that you get the permission to clone my repo and you get write access only for that one branch(?) not sure, as never used this.
image

@Reckless-Satoshi Reckless-Satoshi merged commit 6fc2042 into RoboSats:main Sep 15, 2022
@redphix redphix deleted the fix-non-participant-logic branch September 15, 2022 16:34
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.

None yet

2 participants