Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

make user drawer extensible #9100

Merged
merged 1 commit into from
Oct 22, 2023
Merged

make user drawer extensible #9100

merged 1 commit into from
Oct 22, 2023

Conversation

HexaField
Copy link
Member

@HexaField HexaField commented Oct 20, 2023

Summary

🤖 Generated by Copilot at 7bd326e

Refactor UserDrawer and LinkedProviders components to support hooks.

References

closes #insert number here

Explanation

🤖 Generated by Copilot at 7bd326e

  • Extract LinkedProviders component from UserDrawer component to render identity providers of selected user (link, link, link)
  • Import createHookableFunction utility from common package to allow extending or modifying function components with hooks (link)
  • Wrap UserDrawer component with createHookableFunction utility to enable hooking into its props and rendering (link)
  • Update UserDrawer.tsx file to use LinkedProviders component and createHookableFunction utility (link, link, link, link, link)

🤖 Generated by Copilot at 7bd326e

To make UserDrawer more modular
We extracted LinkedProviders for sure
And used a neat trick
With createHookableFunction
To hook into both components and more

QA Steps

List any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.

Checklist

  • If this PR is still a WIP, convert to a draft
  • When this PR is ready, mark it as "Ready for review"
  • ensure all checks pass
  • Changes have been manually QA'd
  • Changes reviewed by at least 2 approved reviewers

@hanzlamateen hanzlamateen merged commit 319d34d into dev Oct 22, 2023
13 of 14 checks passed
@hanzlamateen hanzlamateen deleted the make-user-drawer-extensible branch October 22, 2023 05:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants