Skip to content

Dashboard fixes #256

Merged
tchapi merged 10 commits intotchapi:mainfrom
lukastrkan:merge/fix-dashboard-memory-leak
May 3, 2026
Merged

Dashboard fixes #256
tchapi merged 10 commits intotchapi:mainfrom
lukastrkan:merge/fix-dashboard-memory-leak

Conversation

@lukastrkan
Copy link
Copy Markdown
Contributor

@lukastrkan lukastrkan commented Apr 18, 2026

Improvements made during the migration from Baikal to Davis:

  • Baikal-compatible route: Added route so existing clients don't need any config changes. Scrapped, does not work reliably
  • Fix out-of-memory on dashboard: Replaced findAll() with count()

Comment thread src/Controller/DAVController.php Outdated
Comment thread src/Controller/DAVController.php Outdated
Comment thread src/Controller/DAVController.php Outdated
Comment thread src/Controller/Admin/DashboardController.php Outdated
@tchapi tchapi self-assigned this Apr 18, 2026
@tchapi tchapi added the enhancement New feature or request label Apr 18, 2026
@lukastrkan lukastrkan changed the title Add Baikal compatible DAV route and optimize data fetching Dashboard fixes May 3, 2026
@lukastrkan
Copy link
Copy Markdown
Contributor Author

Unfortunately, backward compatibility with Baikal did not work reliably, so I scrapped it. However, I added another bugfix: it was impossible to delete an imported user because $entityManager->remove() cannot accept null as an argument.

@lukastrkan lukastrkan requested a review from tchapi May 3, 2026 11:35
@tchapi tchapi merged commit 054a7f8 into tchapi:main May 3, 2026
11 checks passed
@tchapi
Copy link
Copy Markdown
Owner

tchapi commented May 3, 2026

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants