-
Notifications
You must be signed in to change notification settings - Fork 295
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(extension-api): allow extension to register custom navigation route #8837
feat(extension-api): allow extension to register custom navigation route #8837
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think routeid should be prefixed by the extension ID
so we're sure that there is no collapsing
64e47dc
to
e0d5526
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a couple of typos.
Thanks for adding the doc
LGTM
Signed-off-by: axel7083 <42176370+axel7083@users.noreply.github.com>
Signed-off-by: axel7083 <42176370+axel7083@users.noreply.github.com>
Signed-off-by: axel7083 <42176370+axel7083@users.noreply.github.com>
Signed-off-by: axel7083 <42176370+axel7083@users.noreply.github.com>
Signed-off-by: axel7083 <42176370+axel7083@users.noreply.github.com>
Signed-off-by: axel7083 <42176370+axel7083@users.noreply.github.com>
Co-authored-by: Philippe Martin <feloy1@gmail.com> Signed-off-by: axel7083 <42176370+axel7083@users.noreply.github.com>
8f59469
to
81db0f1
Compare
What does this PR do?
Following a discussion we had with @benoitf about #7709. The idea would be to make the extension register a unique navigation route id.
When they create a notification or a task, they could register the navigation route id with some arguments, so the core could navigate to the appropriate route.
Screenshot / video of UI
N/A
What issues does this PR fix or reference?
Required for #7709
How to test this PR?