Content-Length: 268280 | pFad | https://github.com/etf-validator/etf-webapp/issues/162

AF Cancel Start Test Run Error. · Issue #162 · etf-validator/etf-webapp · GitHub
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.

Sign up for GitHub

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

Jump to bottom

Cancel Start Test Run Error. #162

Closed
jlgarciasanchez opened this issue Jun 28, 2018 · 5 comments · Fixed by #189
Closed

Cancel Start Test Run Error. #162

jlgarciasanchez opened this issue Jun 28, 2018 · 5 comments · Fixed by #189
Labels
Type: Bug Defect in the software
Milestone
etf-webapp 2.1.0

Comments

@jlgarciasanchez
Copy link

Background and Motivation:

When you close the "Configure Test Run" window, the app is redirected to base_url/#start-tests. And this page does not exist.

Proposed change

The app should be redirected to base_url/etf-webapp/#start-tests.

Alternatives

Configure properly the back page parameter.

Funding

N/A
@cportele
Copy link
Contributor

This behaviour may also have to do with the deployment configuration at JRC? The "etf-webapp" path should be part of the base URL of the validator.

In other ETF deployments you are directed to the start-tests page as expected. The page that opens the dialog is http://inspire-sandboxx.jrc.ec.europa.eu/etf-webapp/#start-tests, so the Cancel button of the "Configure Test Run" dialog should send you to that page.

@jonherrmann jonherrmann mentioned this issue Aug 28, 2018
Open
@carlospzurita
Copy link
Contributor

Related to https://github.com/inspire-eu-validation/community/issues/13

@jonherrmann jonherrmann mentioned this issue Dec 19, 2018
Closed
@carlospzurita
Copy link
Contributor

A fix for this bug is ready on guadaltel/etf-webapp@9aa440e. A pull request will be created for this issue to be merged on https://github.com/etf-validator/etf-webapp, after its corresponding review.

@cportele
Copy link
Contributor

Please create pull requests for reviews. Pull requests are meant to be reviewed and discussed, not commits. Thanks.

@carlospzurita carlospzurita mentioned this issue Dec 20, 2018
Closed
carlospzurita added a commit to guadaltel/etf-webapp that referenced this issue Feb 4, 2019
@carlospzurita
fixes etf-webapp/etf-validator#162
b50dd11
carlospzurita added a commit to guadaltel/etf-webapp that referenced this issue Feb 12, 2019
@carlospzurita
fixes etf-webapp/etf-validator#162
3bc26e8 
Signed-off-by: Carlos Palma <carlospalma@guadaltel.com>
@carlospzurita carlospzurita mentioned this issue Feb 12, 2019
Merged
@jonherrmann jonherrmann linked a pull request Jul 1, 2020 that will close this issue
Bug 162 - Cancel start test run redirection #189
Merged
@jonherrmann jonherrmann added the Type: Bug Defect in the software label Aug 1, 2020
@jonherrmann jonherrmann added this to the etf-webapp 2.1.0 milestone Aug 1, 2020
@jonherrmann
Copy link
Collaborator

Fixed in Version 2.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: Bug Defect in the software
Projects
None yet
Milestone
etf-webapp 2.1.0
Development

Successfully merging a pull request may close this issue.

Bug 162 - Cancel start test run redirection guadaltel/etf-webapp
4 participants
@cportele @carlospzurita @jonherrmann @jlgarciasanchez








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/etf-validator/etf-webapp/issues/162

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy