-
-
Notifications
You must be signed in to change notification settings - Fork 274
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
Add page parameters to navigation actions #1876
Merged
Merged
Changes from 2 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
7bb9ca0
Add page parameters to navigation actions
apedroferreira b1f0489
Non-working bindings for actions + fix circular dependency hell
apedroferreira c351057
where did this even come from
apedroferreira 99bb4a3
Fix types
apedroferreira d8c8c0b
Working page parameter bindings
apedroferreira 2007d59
Fix and add test
apedroferreira 005f5d1
Merge remote-tracking branch 'origin/master' into add-navigation-page…
apedroferreira 31d818a
Merge remote-tracking branch 'origin/master' into add-navigation-page…
apedroferreira 2d7dcdc
Merge remote-tracking branch 'origin/master' into add-navigation-page…
apedroferreira 07c2cee
Merge remote-tracking branch 'origin/master' into add-navigation-page…
apedroferreira 4d738a0
Fixes for rebase + new page file structure
apedroferreira 30c2075
hehe
apedroferreira 8bcf041
Remove unneeded fixture things
apedroferreira 13bcc23
Fix page name
apedroferreira d28474c
Merge remote-tracking branch 'origin/master' into add-navigation-page…
apedroferreira e6d0e88
Improve test for navigation overall + see if CI passes
apedroferreira 80a4dfe
Better test
apedroferreira 8e4fec1
Attempt to get CI to pass again
apedroferreira 1fcd96e
Remove check to try to debug issue in CI
apedroferreira 1e44bbb
Revert "Remove check to try to debug issue in CI"
apedroferreira e68d8e0
Merge remote-tracking branch 'origin/master' into add-navigation-page…
apedroferreira c1fe290
Another attempt to make CI pass
apedroferreira 4185ed8
Try again
apedroferreira f8d176d
Try using different method
apedroferreira e2ce209
Merge remote-tracking branch 'origin/master' into add-navigation-page…
apedroferreira e057975
Use $ref instead of $$ref everywhere (mystery solved?)
apedroferreira c246b3d
Forgot this one
apedroferreira 55cd8ba
Refactor with ignored cycle to see what changed
apedroferreira ef1980a
Revert some more changes
apedroferreira 4e1c55c
fix lint
apedroferreira 6ff80f1
Merge remote-tracking branch 'origin/master' into add-navigation-page…
apedroferreira 3d255de
Document navigation actions
apedroferreira 558648b
Allow navigating to same page
apedroferreira File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 changes: 1 addition & 1 deletion
2
packages/toolpad-app/src/toolpad/AppEditor/BindingEditor/NavigationActionEditor.tsx
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
109 changes: 0 additions & 109 deletions
109
test/integration/bindings/fixture-navigation/toolpad.yml
This file was deleted.
Oops, something went wrong.
1 change: 1 addition & 0 deletions
1
test/integration/bindings/fixture-navigation/toolpad/.gitignore
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
.generated |
22 changes: 22 additions & 0 deletions
22
test/integration/bindings/fixture-navigation/toolpad/pages/page1/page.yml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
apiVersion: v1 | ||
kind: page | ||
spec: | ||
id: ld03upm | ||
title: page | ||
content: | ||
- component: PageRow | ||
name: pageRow | ||
children: | ||
- component: Button | ||
name: button | ||
props: | ||
onClick: | ||
$$navigationAction: | ||
page: au03sef | ||
parameters: | ||
abc: zyx | ||
def: | ||
$$jsExpression: | | ||
button.content | ||
content: test | ||
display: shell |
11 changes: 11 additions & 0 deletions
11
test/integration/bindings/fixture-navigation/toolpad/pages/page2/page.yml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
apiVersion: v1 | ||
kind: page | ||
spec: | ||
id: au03sef | ||
title: page2 | ||
parameters: | ||
- name: abc | ||
value: "123" | ||
- name: def | ||
value: "456" | ||
display: shell |
6 changes: 6 additions & 0 deletions
6
test/integration/bindings/fixture-navigation/toolpad/theme.yml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
apiVersion: v1 | ||
kind: theme | ||
spec: | ||
palette.mode: light | ||
palette.primary.main: purple | ||
palette.secondary.main: indigo |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
😂