fix: migrate LinkedIn comment API from deprecated v2 to REST#1332
fix: migrate LinkedIn comment API from deprecated v2 to REST#1332swaraj017 wants to merge 9 commits into
Conversation
|
Someone is attempting to deploy a commit to the Listinai Team on Vercel. A member of the Team first needs to authorize it. |
|
@nevo-david Could this bugfix be merged? @swaraj017 You tested this right? |
Yes, I tested it The issue raised earlier was valid, and I have updated the implementation to correctly parse the comment ID from the Tested with the LinkedIn REST API — comment creation and threading are working as expected. |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
|
Will we see this patch in the next update? |
Thanks for the interest @jorgedanisc 🙌 The fix has been implemented and tested against the LinkedIn REST API — comment creation and threading are working as expected. @nevo-david @creyD Also, let me know if any additional validation or edge cases should be covered before merging. |
|
Thanks @github |
|
@nevo-david Would this be able to be merged? |
|
Hi @nevo-david, gentle ping |
|
Hi @swaraj017! Thanks for the PR. Contributions to Postiz are gated behind an application. Please apply at https://contribute.postiz.com/p/postiz and we'll reopen this PR once you're approved. |
|
Contribution-checker quality warning Heuristics that flagged:
If this is a genuine contribution, please add detail to your PR description and tighten the diff scope before reviewers look at it. |
|
Contribution-checker quality warning Heuristics that flagged:
If this is a genuine contribution, please add detail to your PR description and tighten the diff scope before reviewers look at it. |
|
@swaraj017's application for Postiz was approved. Reopening this PR. |
|
Contribution-checker quality warning Heuristics that flagged:
If this is a genuine contribution, please add detail to your PR description and tighten the diff scope before reviewers look at it. |
What kind of change does this PR introduce?
Bug fix
Why was this change needed?
Fixes #1319.
LinkedIn comment creation was still using deprecated v2 endpoints, causing comments and threaded replies to fail. Main post publishing had already been migrated, but the comment flow was missed.
This change updates the comment endpoints to the REST API and adds the required headers for successful requests.
Other information:
Tested against the LinkedIn REST API and verified both comment creation and threaded replies.
Checklist: