Skip to content
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

Execute dynamic sets based on themes #619

Open
AlexBxl opened this issue Jan 7, 2025 · 0 comments
Open

Execute dynamic sets based on themes #619

AlexBxl opened this issue Jan 7, 2025 · 0 comments

Comments

@AlexBxl
Copy link
Contributor

AlexBxl commented Jan 7, 2025

  • Right now if you have a theme node, the backend execution will not take that into consideration. Also theme node definition is not being used on the backend so the graph execution will fail. This causes multiple issues:
  • cannot reference graph tokens output
  • cannot resolve those refs
  • token count in the set view is not being computed.

We should change the useGetAllTokenSets hook to also include the graph execution logic. That means we will have the right result in the ResolveContext, in useAliasList (used for reference dropdown) and other places.

Screenshot 2025-01-08 at 15 09 14
@roppazvan roppazvan changed the title number of tokens in dynamic set not shown Execute dynamic sets based on themes Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant