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
[WIP] Only show SSL table header if there are certificates #15345
Conversation
The inspection completed: 1 new issues |
Refer to this link for build results (access rights to CI server needed): |
very nice 👍 |
So the code there is not really unit testable. We probably need to fix that first before the unit tests can be added here. |
Good stuff, thank you! 👍 |
[WIP] Only show SSL table header if there are certificates
@jancborchardt NEVER merge a PR that is tagged with Developing OR [WIP] OR the next major release. |
Reverted via #15432 |
@rullzer Can I ask you to open a new PR? Thanks |
@rullzer sorry for the mess – only remembered the PR from last week and was focused on reviewing it. :( |
@jancborchardt As requested in #15263
We currently do now have javascript unit tests for the personal page. I'll make a start with that by adding unit tests for this commit later this week. (cc @PVince81)