Closed Cheos137 closed 3 weeks ago
Thanks for the contribution! Before we can merge this, we need @Cheos137 to sign the Salesforce Inc. Contributor License Agreement.
Hi @Cheos137! 👋 Thank you for taking the time to send in this PR and sign the CLA!
This logic is looking solid to me but we might want to include a test to ensure this behavior is correct and doesn't regress in future changes 🙏
Attention: Patch coverage is 0%
with 7 lines
in your changes missing coverage. Please review.
Project coverage is 74.99%. Comparing base (
71ed00d
) to head (a50d102
). Report is 3 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
.../com/slack/api/methods/impl/MethodsClientImpl.java | 0.00% | 7 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
cuz i suspect there might not have gone out a notification from it: yesterday i've added tests to this pr
fix filename & title getting improperly defaulted when uploading multiple files using MethodsClientImpl#filesUploadV2
fixes #1345
Category (place an
x
in each of the[ ]
)Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you agree to those rules.