ToolJet
ToolJet copied to clipboard
Fixed version name sanitization on editing versions
Resolves #4486
deployed https://tj-4570.herokuapp.com/
@surajrane42 Please sign the CLA
@surajrane42 Please fix the lint errors and test cases
@gsmithun4 Signed the CLA & fixed linting errors. Please check
@gsmithun4 @arpitnath I have tried fixing the e2e tests. Can you please check now
@surajrane42 Please rebase https://github.com/surajrane42/ToolJet/pull/1
@gsmithun4 Rebased the branch. Please check
@surajrane42 There is a testcases failing, please fix
@gsmithun4 Sure I will have a look at it & fix it soon
Hey @surajrane42
Thanks for contributing to ToolJet. :raised_hands:
While this PR is under review, you can submit the form to claim your Hacktoberfest swag: https://airtable.com/shr1qR6xhqsWO7LD7
Hey @surajrane42 - Please make the changes suggested by @gsmithun4 so that we can review and merge the PR.
Deployment: https://tooljet-pr-4570.onrender.com Dashboard: https://dashboard.render.com/web/srv-cfksllha6gductgm1p90
Deployment: https://tooljet-pr-4570.onrender.com Dashboard: https://dashboard.render.com/web/srv-cfku8dla49903fhuspsg