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

Fix: PHP web app improvements #2343

Closed
wants to merge 7 commits into from

Conversation

s-martin
Copy link
Collaborator

This addresses some of the concerns of #2342

This is quite untested, so help is appreciated.

@s-martin s-martin added bug help wanted needs testing php legacy_v2 Issues, discussions and PRs related to Version 2.x labels Apr 16, 2024
@s-martin s-martin closed this Apr 16, 2024
@coveralls
Copy link

Pull Request Test Coverage Report for Build 8700182247

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 78.435%

Totals Coverage Status
Change from base Build 8680810264: 0.0%
Covered Lines: 451
Relevant Lines: 575

💛 - Coveralls

@s-martin
Copy link
Collaborator Author

Not feasible approach, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug help wanted legacy_v2 Issues, discussions and PRs related to Version 2.x needs testing php
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants