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

OWTextToColumns: add name migration #266

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

ajdapretnar
Copy link
Contributor

Issue

Old workflows with OWSplit didn't work.

Description of changes

Ensure name migration.

Includes
  • Code changes
  • Tests
  • Documentation

@ajdapretnar
Copy link
Contributor Author

!Note: upon testing with an old workflow, the workflow worked, but the name of the widget remained Split (not Text to Columns as it should be).

@markotoplak
Copy link
Member

I think keeping the name is fine (users can rename widgets in the workflow anyway, and our workflows do not know whether that was the original name or not). The same thing happens with Formula.

@markotoplak markotoplak merged commit fa634fa into biolab:master Nov 24, 2023
12 checks passed
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.

2 participants