feat: remove provider field from member details page - #450
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
💤 Files with no reviewable changes (5)
📝 WalkthroughWalkthroughThe member details view no longer displays provider or connection information. The provider field logic, related tests, and English, French, and Japanese translations were removed. ChangesMember details provider removal
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested labels: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #450 +/- ##
==========================================
- Coverage 90.86% 90.85% -0.01%
==========================================
Files 239 239
Lines 17620 17614 -6
Branches 2467 2573 +106
==========================================
- Hits 16010 16004 -6
Misses 1610 1610 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
@coderabbitai Review the PR changes |
|
✅ Action performedReview finished.
|
Summary
Remove the Provider field from the Member Details component as the
identities/user_stores/identity_providersattribute it will not be available in the API responseWhy
Based on the decision in this Slack discussion, the team agreed to:
identities/user_storesandidentity_providersattribute from the Member Object for theGET /my-org/members/{user_id}endpointsProvider(s)on the MemberDetails Component since that information is no longer returned by the APIWhat
providerfield frombuildMemberDetailFields()utility functionprovidertranslation keys from all locale files (en-US, fr, ja)Packages
packages/corepackages/reactexamplesTesting
Checklist
Contributing
Summary by CodeRabbit