Skip to content

Conversation

@nicokaiser
Copy link
Contributor

@nicokaiser nicokaiser commented Nov 3, 2022

LoadContactsFilesActions adds the "icons" style, which does not exist anymore.

Also, the css/icons.scss file seems to be completely unused, so I removed it. Please correct me if I am wrong.

This should fix #1451

@artonge
Copy link
Collaborator

artonge commented Nov 8, 2022

Hey @nicokaiser, indeed, thanks for the fix :)

Can you fix DCO ? https://github.com/nextcloud/photos/pull/1454/checks?check_run_id=9283127442

@nicokaiser
Copy link
Contributor Author

Can you fix DCO ?

Sorry, I forgot to include the Signed-off-by trailer, fixed it now.

@skjnldsv skjnldsv added the 4. to release Ready to be released and/or waiting for tests to finish label Nov 8, 2022
@artonge artonge merged commit 05c3c2f into nextcloud:master Nov 8, 2022
@artonge
Copy link
Collaborator

artonge commented Dec 12, 2022

/backport to stable25

@backportbot-nextcloud backportbot-nextcloud bot added the backport-request Pending backport by the backport-bot label Dec 12, 2022
@mikey0000
Copy link

Just as an FYI this was causing a 502 within my setup, I did a separate install, same configuration and it only returned 404, not sure why. Took me a few days to find this issue to resolve it.

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

Labels

4. to release Ready to be released and/or waiting for tests to finish backport-request Pending backport by the backport-bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Error 404, css/icons.css not found

4 participants