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 support to transactions with no expense nor order #4748
Conversation
Your Render PR Server URL is https://oc-styleguide-pr-4748.onrender.com. Follow its progress at https://dashboard.render.com/static/srv-bsklq92pp1jjdjar2gr0. |
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/opencollective/opencollective-frontend/o048pxkx7 |
7936fe4
to
8d203f2
Compare
</DetailDescription> | ||
</Box> | ||
)} | ||
{order?.paymentMethod && ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
paymentMethod
should be read at the transaction level I think
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True. I'll implement the missing resolver and fix it altogether.
</Box> | ||
)} | ||
</Flex> | ||
{toAccount.host && order?.paymentMethod && ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm wondering why we should skip this if no paymentMethod
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, this should be an ||
.
8d203f2
to
cfac76e
Compare
Related to opencollective/opencollective#3385
Requires opencollective/opencollective-api#4324