Du änderst die Dateien wieder in ihren Zustand vor dem PR zurück
Das ist mit Git möglich, aber da du mir gesagt hast, dass du ein Git-Anfänger bist, würde ich dir empfehlen, es mit einfachem Kopieren/Einfügen zu machen.
Brisou
8. Dezember 2023 um 11:22
23
Das sollte eigentlich gut sein…
Danke für die Korrekturen, ich habe den gesamten PR gelesen, das ist sehr schön
Es gibt 2 Links, die auf nicht existierende Dokumente verweisen: NextCloud Talk und OpenAI
Meine Review:
master ← jbrisavoine:front-menu-homogene
ouvert 03:30PM - 07 Dec 23 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [ ] If your changes affects code, did your write the tests?
- [x] Are tests passing? (`npm test` on both front/server)
- [x] Is the linter passing? (`npm run eslint` on both front/server)
- [x] Did you run prettier? (`npm run prettier` on both front/server)
- [x] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [ ] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.
- [ ] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
- [ ] If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo](https://github.com/GladysAssistant/v4-website) and the [website](https://gladysassistant.com).
- [ ] Did you add fake requests data for the demo mode (`front/src/config/demo.js`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
Following the request [https://community.gladysassistant.com/t/rendre-les-menus-homogene/8581/1](url):
- Add documentation link on each integration pages.
- Change icons and style for consistency
- Update `src/components/documentation/DeviceConfigurationLink.jsx` to be able to chose the path of the documentation
Before:
<img width="931" alt="2023 12 07_PR_BEFORE" src="https://github.com/GladysAssistant/Gladys/assets/138247436/15d0e725-6f1f-4f5a-9fc0-b0a48762a9c4">
After:
<img width="941" alt="2023 12 07_PR_AFTER" src="https://github.com/GladysAssistant/Gladys/assets/138247436/ed223cca-dfb8-49bb-a442-70ee84c53b78">
Diese Dokumentationen sollten erstellt werden!
Die Dokumentation wurde nicht erstellt.
Soll für OpenAI eine Dokumentation erstellt werden? Du hast es entwickelt.
Ich werde eine Dokumentation schreiben!
Ok, die ChatGPT-Dokumentation ist fertig:
master ← open-ai-documentation
ouvert 02:22PM - 08 Dec 23 UTC
Es ist live auf der Website
Es fehlt nur noch die Nextcloud-Talk-Dokumentation!
Der PR wurde damals erstellt, aber er wurde geschlossen, bevor er gemerged wurde. Ich habe ihn einfach wieder geöffnet
master ← bertrandda:feat/nextcloud-talk-doc
ouvert 08:46PM - 01 Dec 21 UTC
Add Nextcloud Talk documentation fr/en
Screenshot optimized with imageOptim
… Gladys Assistant related PR: https://github.com/GladysAssistant/Gladys/pull/1388
Genau! Der PR wurde vom stalebot geschlossen, bevor das Feature gemerged wurde, das passiert manchmal
Ich habe nachgeschaut, alles gut für mich. Es ist gemerged und wird in 2 Minuten live auf der Website sein
NextCloud Talk live auf der Website!
@Brisou Ich denke, dein PR kann jetzt losgehen
qleg
21. Dezember 2023 um 20:27
33
Ich habe 2 PRs erstellt, um die Links zur Dokumentation für die Sonos- und NextCloud-Integrationen zu korrigieren, die nicht funktionieren.
master ← quentinlegay:fix-dead-link-doc-sonos
ouvert 08:20PM - 21 Dec 23 UTC
The link to the sonos doc was dead because the configurationKey was missing: I'v… e added it.
master ← quentinlegay:fix-dead-link-nextcloud-doc
ouvert 08:21PM - 21 Dec 23 UTC
The link to the nextcloud doc was dead because the documentKey was not correct :… I've fix it.
Danke @qleg für die PRs, es ist gemerged und wird in der nächsten Version von Gladys veröffentlicht!
@alextrovato
Kannst du bitte die Dokumentation für LAN Manager erstellen, wenn du Zeit hast?