Skip to content

Expressions: New not function, extend logic behind component #630

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

Closed
olemartinorg opened this issue Nov 7, 2022 · 0 comments · Fixed by #635
Closed

Expressions: New not function, extend logic behind component #630

olemartinorg opened this issue Nov 7, 2022 · 0 comments · Fixed by #635

Comments

@olemartinorg
Copy link
Contributor

This issue is fairly easy to implement, and could be implemented before the first release of #540. Slightly discussed on Slack between me and @ivarne, we decided on implementing two additional features of the expressions engine:

  1. A new not function. Accepts one boolean (or null) as input, returns !input.
  2. Extend the logic in component to return null if the found component is hidden. This allows for some limited chaining, and makes the component functionally more different from the dataModel lookup (which won't have the same logic).
@olemartinorg olemartinorg self-assigned this Nov 7, 2022
@olemartinorg olemartinorg moved this to 📈 Todo in Team Apps Nov 7, 2022
@olemartinorg olemartinorg moved this from 📈 Todo to 👷 In Progress in Team Apps Nov 7, 2022
@olemartinorg olemartinorg moved this from 👷 In Progress to 🔎 Review in Team Apps Nov 15, 2022
@olemartinorg olemartinorg removed their assignment Nov 15, 2022
Repository owner moved this from 🔎 Review to ✅ Done in Team Apps Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant