-
-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
nextcloud27: 27.1.1 -> 27.1.2 #259348
nextcloud27: 27.1.1 -> 27.1.2 #259348
Conversation
Result of 1 package built:
|
Sucessfully deployed and it's working. But I get the following warning in the admin overview page:
Maybe a change to the nginx config is necessary in the module |
could be this commit nextcloud/server@6200c91 |
Looks like others are having the same issue nextcloud/server#40803 |
I think this doc change is relevant for us https://github.com/nextcloud/documentation/pull/11179/files In fact removing the redirect fixed the admin message for me |
Please pick to fix the warning SuperSandro2000@a5624e4 |
fwiw this patch can't be applied as-is since it'd break nextcloud26 (also 25, but that's almost EOL, so it doesn't count). I started fixing that, but it may take me a day until I can submit the patches :) |
Superseded by #259967. |
Description of changes
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)