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

Improve message for users if HTML report cannot be created from CTL #230

Closed
dstenger opened this issue Sep 6, 2017 · 2 comments
Closed
Assignees
Milestone

Comments

@dstenger
Copy link
Contributor

dstenger commented Sep 6, 2017

  • The alert should not appear when the HTML report is present.
  • Suggestion: Can the alert be replaced by a mechanism which just changes the display of the HTML report to the text currently written in the alert?
    E.g. following is displayed on the result page (no alert):
This test suite does not support the new HTML report and the XML report must be used instead.

See the detailed old test report.
@keshavnangare
Copy link
Contributor

Fixed with this commit: 1be4308

@dstenger
Copy link
Contributor Author

dstenger commented Sep 18, 2017

Test

Test with ets-wms13 and [1]. A new result page is shown as described above and the old report can be accessed.
Test with ets-wfs20 and [2]. New HTML report is shown.

New reporting format if HTML report is not supported has the expected design.

[1] http://cite.deegree.org/deegree-webservices-3.4-RC3/services/wms130?service=WMS&request=GetCapabilities
[2] http://cite.deegree.org/deegree-webservices-3.4-RC3/services/wfs200?service=WFS&request=GetCapabilities

dstenger added a commit that referenced this issue Sep 18, 2017
@dstenger dstenger added this to the 5.0 milestone Oct 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants