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

[JSON Logic] Part 3: Computed string based values #37

Merged
merged 6 commits into from
Sep 13, 2023

Conversation

brennj
Copy link
Collaborator

@brennj brennj commented Aug 23, 2023

Must be merged after #36

Add in x-jsf-logic-computedAttrs (with reference to values defined in x-jsf-logic.computedValues) support.

  • Supports dynamic:
    • x-jsf-errorMessage.minimum/maximum
    • title
    • description
    • x-jsf.presentation.statement.description

src/jsonLogic.js Outdated Show resolved Hide resolved
src/jsonLogic.js Show resolved Hide resolved
src/jsonLogic.js Outdated Show resolved Hide resolved
src/jsonLogic.js Outdated Show resolved Hide resolved
src/jsonLogic.js Show resolved Hide resolved
src/jsonLogic.js Outdated Show resolved Hide resolved
@brennj brennj force-pushed the json-logic-add-computedattrs branch 2 times, most recently from 6b06f89 to d2cc213 Compare August 31, 2023 14:42
@brennj brennj force-pushed the add-string-based-computedAttrs branch 2 times, most recently from 6c036ad to a40da7b Compare September 4, 2023 15:36
johnstonbl01
johnstonbl01 previously approved these changes Sep 11, 2023
Base automatically changed from json-logic-add-computedattrs to main September 12, 2023 07:43
@brennj brennj dismissed johnstonbl01’s stale review September 12, 2023 07:43

The base branch was changed.

author brennj <john.brennan@remote.com> 1694505093 +0200
committer brennj <john.brennan@remote.com> 1694508319 +0200

Release 0.5.0-beta.0

chore: support barebones computedAttrs

chore: fix errors

chore: fix mess ups from rebase

chore: feedback from PR

chore: pass logic down at updateFieldsProperties to prevent bugs

feat: JSON Logic skeleton and plumbing setup

chore: clean up conditional additions

chore: remove const tests

chore: remove dupe file

chore: remove group array stuff

chore: clean up yupschema

chore: clean up helpers a small bit

chore: remove all error handling for now

chore: clean up package-lock

chore: more removing stuff

chore: clean more

chore: support barebones computedAttrs

chore: computed string attributes

chore: fix tests

chore: consistency for curly braces

chore: remove unneeded code for now
@brennj brennj force-pushed the add-string-based-computedAttrs branch from a40da7b to d3f26e2 Compare September 12, 2023 08:50
sandrina-p
sandrina-p previously approved these changes Sep 12, 2023
Copy link
Collaborator

@sandrina-p sandrina-p left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't find anything else odd, let's go p4!

* feat: JSON Logic skeleton and plumbing setup

chore: clean up conditional additions

chore: remove const tests

chore: remove dupe file

chore: remove group array stuff

chore: clean up yupschema

chore: clean up helpers a small bit

chore: remove all error handling for now

chore: clean up package-lock

chore: more removing stuff

chore: clean more

* chore: support barebones computedAttrs

* chore: fix errors

* chore: fix mess ups from rebase

* chore: feedback from PR

* chore: pass logic down at updateFieldsProperties to prevent bugs

* Release 0.5.0-dev.20230901130231

* Revert "Release 0.5.0-dev.20230901130231"

This reverts commit 55ed296.

* feat: JSON Logic skeleton and plumbing setup

chore: clean up conditional additions

chore: remove const tests

chore: remove dupe file

chore: remove group array stuff

chore: clean up yupschema

chore: clean up helpers a small bit

chore: remove all error handling for now

chore: clean up package-lock

chore: more removing stuff

chore: clean more

* chore: support barebones computedAttrs

* chore: computed string attributes

* chore: fix tests

* chore: consistency for curly braces

* chore: remove unneeded code for now

* feat: JSON Logic skeleton and plumbing setup

chore: clean up conditional additions

chore: remove const tests

chore: remove dupe file

chore: remove group array stuff

chore: clean up yupschema

chore: clean up helpers a small bit

chore: remove all error handling for now

chore: clean up package-lock

chore: more removing stuff

chore: clean more

chore: support barebones computedAttrs

chore: computed string attributes

chore: error handling

* chore: fix tests

* chore: review errors

* chore: fix up code for fixtures

* chore: add a bunch of docs to try and make things clearer

* chore: add example to docs

* chore: higher level console check

* chore: use cases to clean up error tests

* chore: add more tests for missing vars

* chore: use switch statement instead

* chore: add code comments why schemas fail

* chore: jsdoc tweaks
brennj and others added 3 commits September 13, 2023 10:35
* feat: JSON Logic skeleton and plumbing setup

chore: clean up conditional additions

chore: remove const tests

chore: remove dupe file

chore: remove group array stuff

chore: clean up yupschema

chore: clean up helpers a small bit

chore: remove all error handling for now

chore: clean up package-lock

chore: more removing stuff

chore: clean more

* chore: support barebones computedAttrs

* chore: fix errors

* chore: fix mess ups from rebase

* chore: feedback from PR

* chore: pass logic down at updateFieldsProperties to prevent bugs

* Release 0.5.0-dev.20230901130231

* Revert "Release 0.5.0-dev.20230901130231"

This reverts commit 55ed296.

* feat: JSON Logic skeleton and plumbing setup

chore: clean up conditional additions

chore: remove const tests

chore: remove dupe file

chore: remove group array stuff

chore: clean up yupschema

chore: clean up helpers a small bit

chore: remove all error handling for now

chore: clean up package-lock

chore: more removing stuff

chore: clean more

* chore: support barebones computedAttrs

* chore: computed string attributes

* chore: fix tests

* chore: consistency for curly braces

* chore: remove unneeded code for now

* feat: JSON Logic skeleton and plumbing setup

chore: clean up conditional additions

chore: remove const tests

chore: remove dupe file

chore: remove group array stuff

chore: clean up yupschema

chore: clean up helpers a small bit

chore: remove all error handling for now

chore: clean up package-lock

chore: more removing stuff

chore: clean more

chore: support barebones computedAttrs

chore: computed string attributes

chore: error handling

* chore: fix tests

* chore: review errors

* chore: fix up code for fixtures

* chore: add a bunch of docs to try and make things clearer

* chore: add example to docs

* chore: higher level console check

* chore: use cases to clean up error tests

* chore: add more tests for missing vars

* chore: use switch statement instead

* chore: add code comments why schemas fail

* feat: JSON Logic skeleton and plumbing setup

chore: clean up conditional additions

chore: remove const tests

chore: remove dupe file

chore: remove group array stuff

chore: clean up yupschema

chore: clean up helpers a small bit

chore: remove all error handling for now

chore: clean up package-lock

chore: more removing stuff

chore: clean more

chore: support barebones computedAttrs

chore: error handling

feat: add more error handling

* chore: changes

* chore: add bad operator handling

* chore: fix bad naming
@brennj
Copy link
Collaborator Author

brennj commented Sep 13, 2023

lets goo

@brennj brennj closed this Sep 13, 2023
@brennj brennj reopened this Sep 13, 2023
@brennj brennj merged commit 6e042ea into main Sep 13, 2023
4 checks passed
@brennj brennj deleted the add-string-based-computedAttrs branch September 13, 2023 09:48
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

Successfully merging this pull request may close these issues.

None yet

4 participants