Skip to content

Fix/allow force get user info - #219

Merged
smarcet merged 3 commits into
mainfrom
fix/allow-force-getUserInfo
Apr 21, 2026
Merged

Fix/allow force get user info#219
smarcet merged 3 commits into
mainfrom
fix/allow-force-getUserInfo

Conversation

@santipalenque

@santipalenque santipalenque commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

Allowing force pull on getUserInfo : https://app.clickup.com/t/86b8n819k

Summary by CodeRabbit

  • Chores

    • Updated package version to 5.0.10-beta.1.
  • New Features

    • Added capability to force refresh of user information, allowing bypass of cached data when needed.

@coderabbitai

coderabbitai Bot commented Apr 15, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 87d7d982-a921-4e20-870e-c88f72b0bc22

📥 Commits

Reviewing files that changed from the base of the PR and between 7270594 and af581c3.

📒 Files selected for processing (2)
  • package.json
  • src/components/security/actions.js

📝 Walkthrough

Walkthrough

The PR updates the package version to a beta release and modifies the getUserInfo thunk creator to accept a new optional forcePull parameter, enabling callers to bypass cached member data and force a fresh fetch when needed.

Changes

Cohort / File(s) Summary
Package Version
package.json
Updated version from 5.0.8 to 5.0.10-beta.1.
Cache Control Enhancement
src/components/security/actions.js
Added forcePull parameter (default false) to getUserInfo thunk. Modified control flow to only skip fetching when both member != null and !forcePull, allowing callers to force fresh data retrieval.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A version hops forward, beta in sight,
With forcePull now guiding the fetch just right,
No more stale cached whispers for those who dare,
Fresh member data dances through the air! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: adding a forcePull option to the getUserInfo function to allow bypassing cached state and forcing a fresh data fetch.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/allow-force-getUserInfo

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.

@smarcet
smarcet merged commit 92a0516 into main Apr 21, 2026
5 checks passed
@santipalenque
santipalenque deleted the fix/allow-force-getUserInfo branch August 26, 2026 16:27
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.

2 participants