Creating draft PR for working on issue 25011
Hey, I just made a Pull Request!
@vinzscam @piatkiewicz , I have created this PR for working on the issue: https://github.com/backstage/backstage/issues/25011
Will fill up the details as I continue to work. Thanks for the support so far guys!
:heavy_check_mark: Checklist
Thanks for the contribution! All commits need to be DCO signed before they are reviewed. Please refer to the the DCO section in CONTRIBUTING.md or the DCO status for more info.
Update: found the issue in the Router component, working on resolving it soon to fix issue: #25011
Hi!
For issue #25011
I'm currently working on setting up routing for some components within a <MemoryRouter> tag. Specifically, I'm trying to provide the correct routing context for the AggregateMembersToggle and Default
While it makes sense to use GroupProfileCard as an element within the route, it leads to issues with permission hooks
Any feedback or direction on:
-
The appropriate routing tags and structure within <MemoryRouter> to provide context.
-
Is it the right to integrate GroupProfileCard? If yes how we do it without permission issues.
-
Any additional components or mock implementations needed to resolve these issues.
@Rugvip @camilaibs @iamEAP @freben @vinzscam , I would really appreciate any sort of help on this.
Hi! For issue #25011 I'm currently working on setting up routing for some components within a tag. Specifically, I'm trying to provide the correct routing context for the AggregateMembersToggle and Default
While it makes sense to use GroupProfileCard as an element within the route, it leads to issues with permission hooks
Any feedback or direction on:
- The appropriate routing tags and structure within to provide context.
- Is it the right to integrate GroupProfileCard? If yes how we do it without permission issues.
- Any additional components or mock implementations needed to resolve these issues.
@Rugvip @camilaibs @iamEAP @freben @vinzscam , I would really appreciate any sort of help on this.
Replied here: https://github.com/backstage/backstage/issues/25011#issuecomment-2211681184
Hi all, I am marking this PR ready for review, this fixes issue #25011
As suggested by @camilaibs , I have added the TestApp for giving missing routing context and removed the MemoryRouter to avoid conflicts.
The issue is fixed, the Group Member List Card components are properly coming up now.
Thanks @vinzscam @piatkiewicz for help
Tagging a few maintainers for review @Rugvip @mtlewis @benjdlambert
P.S: I have added the signoff, I still don't know why DCO sign-off check for the commit is failing, rest all is good. Please let me know if anything else is needed to be done, cheers!
Refrence images added.