-
Notifications
You must be signed in to change notification settings - Fork 10
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
improvements from 35 #857
Merged
Merged
improvements from 35 #857
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
Contributor
keevan
commented
Dec 22, 2023
- feat: add connector SQL step
- fix: allow values to be corrected in bad states
- feat: add set multiple variables step type
- fix: warning logs now display and are more legible
- fix: add copy file error handling for directory creation
- feat: run name and id are now available as expressable vars
- fix: SQL trait handling of integer values and improve error message
- fix: update email flow step to support it as a final step
- fix: reader csv error to be more correct for header/fields count alignment
- fix: include row number in csv reader error
- perf: cache sftp object per identity to speed up subsequent actions
- feat: add an option to continue processing the csv file on known parse issues
- build: bump version
Instead of allowing the state to be not recoverable, this will instead fail to render the available variables section only.
Similar to set_variable but this allows multiple values to be set on an existing or new path. This simplifies initialisation of multiple variables per instance run, such as counters used for reporting.
- The background with white text was difficult to read (low contrast) - The warning regex previously in place was incorrect
Also tweaked comment on string format of run based output logs.
…nment It is more correct to expect the number of fields to match the number of headers, not the other way around necessarily.
…e issues The only one is currently a count mismatch on header and the row fields
Peterburnett
approved these changes
Dec 22, 2023
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.
Approved to merge once CI green
Easily update user profile fields given a user id and a yaml formatted list of fields to update
- Add step in upgrade to update existing log files
keevan
force-pushed
the
improvements-from-35
branch
from
December 22, 2023 04:00
444c1a9
to
f1129d4
Compare
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.