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

Update main to development branch #2

Merged
merged 4 commits into from
Jun 2, 2024
Merged

Conversation

TungstnBallon
Copy link
Collaborator

This updated main to the latest developent version

  1. rebases onto latest upstream jayvee
  2. add transforms
  3. add fix typscript crash

closes #1

@TungstnBallon TungstnBallon self-assigned this Jun 2, 2024
georg-schwarz and others added 3 commits June 2, 2024 11:29
from:
ad744b78dcef3bb1bf5ac7dd4c8c77a361322343
to:
8bf5053a7081e57fca1a971d46b490851dc03d1f
1. The structuredClone() function from javascript loses type
   information, so it's not suitable for a typescript project.
   I solved this by converting the old values into JSON, parsing that
   into a new object and then manually asserting the types again.

2. The output columns created during a transform exectution had the name
   of the output variable from the transform block, not the specified
   `outputColumnName` from the TableTransform block.
   This was due to a bug in TsTable.addColumn, which did not ensure the
   supplied name was the same as the actual columnname
@TungstnBallon TungstnBallon merged commit e06b657 into main Jun 2, 2024
1 check passed
@TungstnBallon TungstnBallon deleted the other-type-system branch June 2, 2024 09:39
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.

[BUG] typescript backend crashes
2 participants