update and cleanup of pages #479

Merged
meaz merged 6 commits from update_cleanup into main 2024-04-25 11:10:42 +02:00
Owner
See https://git.disroot.org/Disroot/Disroot-Project/issues/842
antilopa was assigned by meaz 2024-04-23 21:43:59 +02:00
avg_joe was assigned by meaz 2024-04-23 21:43:59 +02:00
fede was assigned by meaz 2024-04-23 21:43:59 +02:00
muppeth was assigned by meaz 2024-04-23 21:43:59 +02:00
meaz added 2 commits 2024-04-23 21:44:00 +02:00
floss4good reviewed 2024-04-24 01:15:20 +02:00
@ -28,3 +26,3 @@
- _nextcloud-talk
- _nextcloud-sweep
- _nextcloud-checksum
- _nextcloud-cookbook
Contributor

Shouldn't you include also _nextcloud-cospend?

Shouldn't you include also `_nextcloud-cospend`?
meaz marked this conversation as resolved
@ -17,3 +17,3 @@
# Anonymous multi search engine platform
Disroot's Search is a search engine like Google, DuckDuckGo, Qwant, powered by **Searx**. What makes it unique from others is that it is a metasearch engine, aggregating the results of other search engines while not storing information about its users.
Disroot's Search is a search engine like Google, DuckDuckGo, Qwant, powered by **Searxng**. What makes it unique from others is that it is a metasearch engine, aggregating the results of other search engines while not storing information about its users.
Contributor

Official spelling is SearXNG.

And the files from pages/03.services/11.search/_search-tech should also be updated.

Official spelling is `SearXNG`. And the files from pages/03.services/11.search/_search-tech should also be updated.
meaz marked this conversation as resolved
@ -23,3 +23,3 @@
Disroot Search: [https://search.disroot.org](https://search.disroot.org)
Source code: [https://github.com/asciimoo/searx](https://github.com/asciimoo/searx)
Source code: [https://github.com/searxng/searxng](https://github.com/searxng/searxng)
Contributor

The same should be changed also for DE, ES and FR.

The same should be changed also for DE, ES and FR.
meaz marked this conversation as resolved
Contributor

Does it make any sense to keep duplicate screenshots per language, like the below mentioned ones, even if there is nothing translated?

  • pages/03.services/10.upload/_lufi-how/de/lufi-drop.png
  • pages/03.services/10.upload/_lufi-how/en/lufi-drop.png
Does it make any sense to keep duplicate screenshots per language, like the below mentioned ones, even if there is nothing translated? - pages/03.services/10.upload/_lufi-how/de/lufi-drop.png - pages/03.services/10.upload/_lufi-how/en/lufi-drop.png
meaz added 1 commit 2024-04-24 09:36:39 +02:00
Author
Owner

Does it make any sense to keep duplicate screenshots per language, like the below mentioned ones, even if there is nothing translated?

  • pages/03.services/10.upload/_lufi-how/de/lufi-drop.png
  • pages/03.services/10.upload/_lufi-how/en/lufi-drop.png

No it doesn't make sense indeed.

Thanks for all the points, I resolved all of them @floss4good

> Does it make any sense to keep duplicate screenshots per language, like the below mentioned ones, even if there is nothing translated? > - pages/03.services/10.upload/_lufi-how/de/lufi-drop.png > - pages/03.services/10.upload/_lufi-how/en/lufi-drop.png No it doesn't make sense indeed. Thanks for all the points, I resolved all of them @floss4good
meaz added 1 commit 2024-04-24 09:44:15 +02:00
floss4good reviewed 2024-04-24 11:09:30 +02:00
@ -0,0 +1,12 @@
---
Contributor

This addition should probably be followed by an update of the last_modified date from translation banner (for all languages).

This addition should probably be followed by an update of the `last_modified` date from translation banner (for all languages).
meaz marked this conversation as resolved
@ -19,3 +19,3 @@
Disroot Cryptpad: [cryptpad.disroot.org](https://cryptpad.disroot.org)
Disroot CryptPad: [cryptpad.disroot.org](https://cryptpad.disroot.org)
Projekt Website: [https://cryptpad.fr](https://cryptpad.fr)
Contributor

Also here, please: cryptpad.fr → cryptpad.org

Also here, please: cryptpad.fr → cryptpad.org
meaz marked this conversation as resolved
@ -21,3 +21,3 @@
Project homepage: [https://cryptpad.fr](https://cryptpad.fr)
Project homepage: [https://cryptpad.org](https://cryptpad.org)
Source code: [https://github.com/xwiki-labs/cryptpad](https://github.com/xwiki-labs/cryptpad)
Contributor

The source code URL is now https://github.com/cryptpad/cryptpad
(same for all languages)

The source code URL is now https://github.com/cryptpad/cryptpad (same for all languages)
meaz marked this conversation as resolved
@ -1,23 +0,0 @@
---
Contributor

I did not tested this branch locally, but I was expected to see the _cryptpad-diagram partial included (for all languages) rather than this file being deleted. Am I missing something?!

I did not tested this branch locally, but I was expected to see the `_cryptpad-diagram` partial included (for all languages) rather than this file being deleted. Am I missing something?!
Author
Owner

my mistake. @fede restored it already.

my mistake. @fede restored it already.
meaz marked this conversation as resolved
@ -18,2 +18,4 @@
Searx è un metamotore, significa che aggrega i risultati di altri motori di ricerca e lo fa senza salvare informazioni sugli utenti.
Disroot's Search è un motore di ricerca come Google, DuckDuckGo e Qwant, alimentato da **SearXNG**. Ciò che lo rende unico rispetto agli altri è che si tratta di un motore di metaricerca, che aggrega i risultati di altri motori di ricerca senza memorizzare alcuna informazione sui suoi utenti.
Contributor

The last_modified date from translation banner should probably be updated.

The `last_modified` date from translation banner should probably be updated.
meaz marked this conversation as resolved
Contributor

No it doesn't make sense indeed.

Since pages/03.services/10.upload/_lufi-how/de/lufi-drop.png was deleted, pages/03.services/10.upload/_lufi-how/2col.de.md should now be updated to use the en/lufi-drop.png screenshot.

BTW: there are other similar cases (e.g. pages/03.services/03.nextcloud/_nextcloud-cals/de), but maybe it would be better to handle them into separate PR/s, since this one is getting quite big and hard to review; or just make sure to handle them while working on the new website.

> No it doesn't make sense indeed. Since pages/03.services/10.upload/_lufi-how/de/lufi-drop.png was deleted, pages/03.services/10.upload/_lufi-how/2col.de.md should now be updated to use the en/lufi-drop.png screenshot. BTW: there are other similar cases (e.g. pages/03.services/03.nextcloud/_nextcloud-cals/de), but maybe it would be better to handle them into separate PR/s, since this one is getting quite big and hard to review; or just make sure to handle them while working on the new website.
fede added 1 commit 2024-04-24 14:25:34 +02:00
fede approved these changes 2024-04-24 14:35:51 +02:00
Dismissed
meaz added 1 commit 2024-04-24 22:48:01 +02:00
Author
Owner

BTW: there are other similar cases (e.g. pages/03.services/03.nextcloud/_nextcloud-cals/de), but maybe it would be better to handle them into separate PR/s, since this one is getting quite big and hard to review; or just make sure to handle them while working on the new website.

As I was on it I corrected those also :) So we're good normally. If you don't see anything left I might have overlooked, let me know so that I can merge this @floss4good Thanks again!

> BTW: there are other similar cases (e.g. pages/03.services/03.nextcloud/_nextcloud-cals/de), but maybe it would be better to handle them into separate PR/s, since this one is getting quite big and hard to review; or just make sure to handle them while working on the new website. As I was on it I corrected those also :) So we're good normally. If you don't see anything left I might have overlooked, let me know so that I can merge this @floss4good Thanks again!
fede approved these changes 2024-04-25 00:37:20 +02:00
meaz merged commit ec69c7d177 into main 2024-04-25 11:10:42 +02:00
meaz deleted branch update_cleanup 2024-04-25 11:10:42 +02:00
meaz referenced this pull request from a commit 2024-04-25 11:10:44 +02:00
Contributor

If you don't see anything left I might have overlooked, let me know so that I can merge this @floss4good Thanks again!

You're welcome, @meaz. And sorry for not being able to check this earlier.
If I have any other suggestion I will open a new issue/PR.

> If you don't see anything left I might have overlooked, let me know so that I can merge this @floss4good Thanks again! You're welcome, @meaz. And sorry for not being able to check this earlier. If I have any other suggestion I will open a new issue/PR.
Sign in to join this conversation.
No description provided.