nextcloud
Support Nextcloud 30, update CSP header
Closes #1333 It probably makes most sense to review this commit-by-commit.
cc @come-nc @susnux @tacruc
Btw I've seen that the CI failed because of a missing Signed-off-by, will fix it up on the next iteration.
Hi @Ma27, Thanks a lot for your contribution I appreciate it a lot. Unfortunately I'm currently quite busy. Right now I'm sitting in a train and wanted to use the time to update and merge this. Unfortunately my laptop run out of battery and the plugs are not working. So I'm stuck with my phone.
Could you update the Test workflow such that the tests are run on NC30? https://github.com/nextcloud/maps/blob/master/.github/workflows/test.yml
Hi @tacruc! No worries at all!
Pushed a commit that does that. Not sure if it's completely correct though, I usually work with other CI systems :upside_down_face: However, I think you should be able to push to my fork's branch as well, right?
Thanks a lot. In theory I should be able to push to your branch. In practice I didn't find a good git client for Android. Do you know one?
Can you remove v28 and v29 from testing and make the DCO, happy? I think then we can go for a merge.
@Ma27, there is a nightly build with your PR https://apps.nextcloud.com/apps/maps