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

Follow up to PSR-4 migration: App.php to application migration #223

Merged
merged 3 commits into from
Jun 19, 2023

Conversation

burned42
Copy link
Contributor

@burned42 burned42 commented Jun 19, 2023

I noticed that one of the files I moved in #222 was the app.php which contains the class Application. I took a look at the documentation at https://docs.nextcloud.com/server/latest/developer_manual/app_development/bootstrap.html which says that the app.php is deprecated and I think the code inside should be adjusted as well.

I used said documentation and also took a look at different applications under https://github.com/nextcloud e.g.

I deactivated and removed keeweb from my instance, built keeweb as described in README, uploaded, extracted and activated it and it still works for me and I don't get any log entries on debug loglevel.

Related to #219 (works for me without issues, so this time the issue might be resolved for real)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants