Skip to content

Conversation

@michelle0927
Copy link
Collaborator

@michelle0927 michelle0927 commented Sep 26, 2025

Resolves #18400

Summary by CodeRabbit

  • Documentation

    • Clarified Post ID field description to include required URN formats and where to find them.
    • Standardized wording in action descriptions (using “documentation” links).
  • Chores

    • Bumped versions for multiple LinkedIn actions and one source to reflect metadata updates.
    • Updated LinkedIn package version.
  • Refactor

    • Internal cleanup to align request configuration with current platform guidance (no user-facing behavior changes).

@vercel
Copy link

vercel bot commented Sep 26, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
pipedream-docs Ignored Ignored Sep 26, 2025 7:49pm
pipedream-docs-redirect-do-not-edit Ignored Ignored Sep 26, 2025 7:49pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 26, 2025

Walkthrough

Version fields were incremented across multiple LinkedIn actions and one source. Two actions updated description text. The LinkedIn app removed the X-Restli-Protocol-Version header and clarified Post Id description. The package version was bumped. No control-flow or runtime logic changed in actions or source.

Changes

Cohort / File(s) Summary of Changes
Actions: version bumps only
components/linkedin/actions/create-comment/create-comment.mjs, .../create-image-post-organization/create-image-post-organization.mjs, .../create-image-post-user/create-image-post-user.mjs, .../create-like-on-share/create-like-on-share.mjs, .../create-text-post-organization/create-text-post-organization.mjs, .../create-text-post-user/create-text-post-user.mjs, .../fetch-ad-account/fetch-ad-account.mjs, .../get-current-member-profile/get-current-member-profile.mjs, .../get-member-profile/get-member-profile.mjs, .../get-multiple-member-profiles/get-multiple-member-profiles.mjs, .../get-org-member-access/get-org-member-access.mjs, .../get-organization-access-control/get-organization-access-control.mjs, .../get-organization-administrators/get-organization-administrators.mjs, .../get-profile-picture-fields/get-profile-picture-fields.mjs, .../retrieve-comments-shares/retrieve-comments-shares.mjs, .../search-organization/search-organization.mjs
Incremented exported version fields; no logic, control-flow, or API changes.
Actions: description text + version bump
components/linkedin/actions/delete-post/delete-post.mjs, .../retrieve-comments-on-comments/retrieve-comments-on-comments.mjs
Updated description strings and bumped version. No logic changes.
Source: version bump
components/linkedin/sources/new-organization-post-created/new-organization-post-created.mjs
Incremented exported version; no logic changes.
LinkedIn app config
components/linkedin/linkedin.app.mjs
Clarified Post Id description (URN formats). Removed X-Restli-Protocol-Version from _getHeaders(); retained Content-Type and Linkedin-Version.
Package metadata
components/linkedin/package.json
Bumped package version from 1.2.2 to 1.2.3.

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • lcaresia
  • dannyroosevelt

Poem

A nibble of versions, hop hop hooray,
Headers trimmed lightly, we’re on our way.
Descriptions now clearer, URNs in sight,
Carrots for correctness, shipped just right.
Thump-thump the merge—what a tidy delight! 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (3 warnings, 1 inconclusive)
Check name Status Explanation Resolution
Linked Issues Check ⚠️ Warning The changeset fails to address any of the bug fixes listed in issue #18400, as it only increments version numbers and updates metadata without unpublishing the problematic access control component or fixing the delete-post and retrieve-comments on comments errors. Implement the actual bug fixes from issue #18400 by unpublishing the access control component and correcting the delete-post and retrieve-comments-on-comments actions.
Out of Scope Changes Check ⚠️ Warning All modifications in this PR are version and metadata updates that do not relate to the bug corrections defined in the linked issue, so almost every change is unrelated to the stated objectives. Move the version bumps to a separate metadata update PR and focus this pull request on resolving the linked issue’s functional bugs.
Description Check ⚠️ Warning The pull request description only includes a “Resolves #18400” reference and does not follow the required template, omitting the “## WHY” section and explanation of the rationale behind the changes. Please update the description to include the “## WHY” section from the template and provide context on why these changes are necessary.
Title Check ❓ Inconclusive The title is broad and refers generally to LinkedIn component updates without specifying the primary purpose or key change; it does not clearly summarize the main changeset or bug fixes being targeted. Consider revising the title to clearly reflect the main objective or specific change, such as fixing LinkedIn component bugs or bumping version metadata for LinkedIn actions.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch issue-18400

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1f8bd63 and d4357c0.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (21)
  • components/linkedin/actions/create-comment/create-comment.mjs (1 hunks)
  • components/linkedin/actions/create-image-post-organization/create-image-post-organization.mjs (1 hunks)
  • components/linkedin/actions/create-image-post-user/create-image-post-user.mjs (1 hunks)
  • components/linkedin/actions/create-like-on-share/create-like-on-share.mjs (1 hunks)
  • components/linkedin/actions/create-text-post-organization/create-text-post-organization.mjs (1 hunks)
  • components/linkedin/actions/create-text-post-user/create-text-post-user.mjs (1 hunks)
  • components/linkedin/actions/delete-post/delete-post.mjs (1 hunks)
  • components/linkedin/actions/fetch-ad-account/fetch-ad-account.mjs (1 hunks)
  • components/linkedin/actions/get-current-member-profile/get-current-member-profile.mjs (1 hunks)
  • components/linkedin/actions/get-member-profile/get-member-profile.mjs (1 hunks)
  • components/linkedin/actions/get-multiple-member-profiles/get-multiple-member-profiles.mjs (1 hunks)
  • components/linkedin/actions/get-org-member-access/get-org-member-access.mjs (1 hunks)
  • components/linkedin/actions/get-organization-access-control/get-organization-access-control.mjs (1 hunks)
  • components/linkedin/actions/get-organization-administrators/get-organization-administrators.mjs (1 hunks)
  • components/linkedin/actions/get-profile-picture-fields/get-profile-picture-fields.mjs (1 hunks)
  • components/linkedin/actions/retrieve-comments-on-comments/retrieve-comments-on-comments.mjs (1 hunks)
  • components/linkedin/actions/retrieve-comments-shares/retrieve-comments-shares.mjs (1 hunks)
  • components/linkedin/actions/search-organization/search-organization.mjs (1 hunks)
  • components/linkedin/linkedin.app.mjs (1 hunks)
  • components/linkedin/package.json (1 hunks)
  • components/linkedin/sources/new-organization-post-created/new-organization-post-created.mjs (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: pnpm publish
🔇 Additional comments (16)
components/linkedin/actions/get-organization-administrators/get-organization-administrators.mjs (1)

7-7: Version metadata update looks good

Line 7 cleanly increments the action version without touching runtime behavior. No concerns here.

components/linkedin/actions/create-comment/create-comment.mjs (1)

7-7: Version metadata update looks good

Line 7 simply bumps the action version as expected; everything else remains stable.

components/linkedin/actions/create-text-post-user/create-text-post-user.mjs (1)

8-8: Version metadata update looks good

Line 8 correctly advances the action’s version. No additional changes detected.

components/linkedin/actions/get-profile-picture-fields/get-profile-picture-fields.mjs (1)

8-8: Version metadata update looks good

Line 8 updates the version field in isolation, which is fine.

components/linkedin/actions/retrieve-comments-shares/retrieve-comments-shares.mjs (1)

7-7: Version metadata update looks good

Line 7’s version increment is consistent with the release strategy.

components/linkedin/actions/create-like-on-share/create-like-on-share.mjs (1)

7-7: Version metadata update looks good

Line 7 advances the version without altering logic—no issues spotted.

components/linkedin/actions/get-organization-access-control/get-organization-access-control.mjs (1)

7-7: Version metadata update looks good

Line 7 increments the action version appropriately; the remainder of the file is unchanged.

components/linkedin/actions/get-member-profile/get-member-profile.mjs (1)

7-7: Version metadata update looks good

Line 7’s version bump is straightforward and causes no behavioral change.

components/linkedin/actions/search-organization/search-organization.mjs (1)

7-7: Version bump is consistent

Increment aligns with the broader LinkedIn package release; no functional issues spotted.

components/linkedin/actions/create-image-post-user/create-image-post-user.mjs (1)

10-10: Patch version looks good

Metadata bump only; implementation remains stable.

components/linkedin/actions/get-current-member-profile/get-current-member-profile.mjs (1)

7-7: Descriptor update confirmed

Version increment matches the package-level bump.

components/linkedin/actions/fetch-ad-account/fetch-ad-account.mjs (1)

7-7: Action metadata bump approved

No logic changes; version tracks with the rest of the release.

components/linkedin/actions/create-text-post-organization/create-text-post-organization.mjs (1)

8-8: Version increment acknowledged

Patch release metadata looks correct for this action.

components/linkedin/package.json (1)

3-3: Package version bump makes sense

1.2.3 lines up with the action patch releases.

components/linkedin/sources/new-organization-post-created/new-organization-post-created.mjs (1)

12-12: Source version bump accepted

Patch version tracks with the component updates.

components/linkedin/actions/create-image-post-organization/create-image-post-organization.mjs (1)

10-10: Metadata update verified

Version 1.0.5 aligns with the coordinated release set.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@michelle0927 michelle0927 marked this pull request as ready for review September 26, 2025 19:51
Copy link
Collaborator

@GTFalcao GTFalcao left a comment

Choose a reason for hiding this comment

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

LGTM!

@vunguyenhung vunguyenhung merged commit 34c7a1a into master Sep 29, 2025
10 checks passed
@vunguyenhung vunguyenhung deleted the issue-18400 branch September 29, 2025 01:20
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.

[BUG] LinkedIn - Components updates

4 participants