-
-
Notifications
You must be signed in to change notification settings - Fork 233
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
. #712
Closed
parth-sd
wants to merge
63
commits into
harttle:master
from
SpotDraft:SPD-23149-add-sum-filter-in-liquidjs-to-get-the-sum-of-a-dynamic-table-column-or-an-array
Closed
. #712
parth-sd
wants to merge
63
commits into
harttle:master
from
SpotDraft:SPD-23149-add-sum-filter-in-liquidjs-to-get-the-sum-of-a-dynamic-table-column-or-an-array
Conversation
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
Fix rounding off issue
* added validator for expressions and also added tests for it * added tests and fixed bugs * fix
* added moment library * allow date and term calculation in liquid * refactor addDuration and subtractDates * return values directly * Fix duration in days calculation * add tests * Maintain consistency in naming decorators * use existing decorators to calculate date term calculation * spell check * remove functions that are not needed * WIP: tests updated * Co-authored-by: Madhav Bhagat <[email protected]> tests for add duration and subrtract date * return duration instead of no of days * add default argument for getObjectValues function * handle from_date greater than to_date case * update tests for negative value of durationInDays * support for adding durartion and tests for the same * added tests for assign * throw error if either value or type is incorrcect in duration calculation. Co-authored-by: Sumanth Subbarao <[email protected]>
Fix type case issue
Update with upstream
Render issue fix
…tation (#16) Co-authored-by: Shubham <[email protected]>
* refactored code for null values and added tests * added isNan in valid number check Co-authored-by: Shubham <[email protected]>
Co-authored-by: Shubham <[email protected]>
Co-authored-by: Shubham Uniyal <[email protected]>
Co-authored-by: Shubham Uniyal <[email protected]>
* fixed redundant tests * Added tests and made changes in logic for null computation Co-authored-by: Shubham <[email protected]>
* added separate tag for returning variable value from appending vars * added tests Co-authored-by: Shubham <[email protected]>
Co-authored-by: Shubham <[email protected]>
* Add test and method to get dependecy tree for a single assign tag * Change the return type of parseAssign * Handle if else conditions * add test for complex template * Add function to transverse the graph to get affected variables * Add comments * Obfuscate test vars
…check Added function and tests for cyclic dependency check
…27) Co-authored-by: Sanket Mishra <[email protected]>
* added null operation handling for different operations * Added tests for liquid and updated code * Added comments * updated comments Co-authored-by: Shubham <[email protected]>
* add new tag- parseAssign * remove .only from describe in parseAssign test Co-authored-by: Sanket Mishra <[email protected]>
Co-authored-by: Sanket Mishra <[email protected]>
…31) * support comparision of commonform data types directly using operators * remove country_code check for phone number Co-authored-by: Sanket Mishra <[email protected]>
Co-authored-by: Sanket Mishra <[email protected]>
Co-authored-by: Sanket Mishra <[email protected]>
feat: add new dependency method to get all vars that are assigned a value
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.