Skip to content
This repository was archived by the owner on Jan 10, 2026. It is now read-only.

Fix not redirecting to Pipeline List after importing if the import URL contains trailing slash#778

Merged
iannbing merged 2 commits intodevfrom
fix/import-url-regex
Mar 10, 2022
Merged

Fix not redirecting to Pipeline List after importing if the import URL contains trailing slash#778
iannbing merged 2 commits intodevfrom
fix/import-url-regex

Conversation

@iannbing
Copy link
Copy Markdown
Contributor

@iannbing iannbing commented Mar 9, 2022

Description

The regular expression for the import_url should take into account the trailing slash.

Fixes: #743

the regex should take into account the trailing backslash in import_url
@iannbing iannbing added the improvement An improvement or enhancement to an existing feature. label Mar 9, 2022
@cypress
Copy link
Copy Markdown

cypress Bot commented Mar 9, 2022



Test summary

88 0 0 0


Run details

Project orchest
Status Passed
Commit 58a7582 ℹ️
Started Mar 9, 2022 9:09 PM
Ended Mar 9, 2022 10:01 PM
Duration 52:27 💡
OS Linux Ubuntu - 20.04
Browser Chrome 88

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@iannbing iannbing requested a review from yannickperrenet March 9, 2022 18:29
@iannbing iannbing temporarily deployed to integration-tests March 9, 2022 20:13 Inactive
Copy link
Copy Markdown
Contributor

@yannickperrenet yannickperrenet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The fix makes sense to me 👍

@iannbing iannbing merged commit 74b5602 into dev Mar 10, 2022
@iannbing iannbing deleted the fix/import-url-regex branch March 10, 2022 10:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

improvement An improvement or enhancement to an existing feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants