[bug] Assumes all apps can be cloned from the `nextcloud` org (including suggested fix)
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
Direzione di ricerca
Inizia in bootstrap.sh alla riga indicata intorno alla 158 e segui il modo in cui vengono clonate le app aggiuntive richieste. Esegui ./bootstrap.sh music prima e dopo la modifica; il lavoro è completato quando l’app Music viene recuperata da https://github.com/nc-music/music invece di assumere che appartenga all’organizzazione nextcloud.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The Music app's repo is at https://github.com/nc-music/music but the script assumes all extra apps prompted by the user are under the nextcloud org, so running ./bootstrap.sh music fails. The following is the line doing this:
Maybe supporting a list of known apps can help, this also makes it easier to contribute extra apps. For example:
function clone_app() {
TARGET=workspace/server/apps-extra/"$1"
if [ -d "$TARGET"/.git ]; then
echo "🆗 App $1 is already installed." | indent
return
fi
(
echo "🌏 Fetching $1"
(git clone ${APPS_CLONE_PARAMS[@]+"${APPS_CLONE_PARAMS[@]}"} "$2".git "$TARGET" 2>&1 | indent_cli &&
echo "✅ $1 installed") ||
echo "❌ Failed to install $1"
) | indent
}
function install_app() {
case "$1" in
viewer) clone_app $1 "https://github.com/orgs/nextcloud/viewer" ;;
recommendations) clone_app $1 "https://github.com/orgs/nextcloud/recommendations" ;;
files_pdfviewer) clone_app $1 "https://github.com/orgs/nextcloud/files_pdfviewer" ;;
profiler) clone_app $1 "https://github.com/orgs/nextcloud/profiler" ;;
hmr_enabler) clone_app $1 "https://github.com/orgs/nextcloud/hmr_enabler" ;;
circles) clone_app $1 "https://github.com/orgs/nextcloud/circles" ;;
globalsiteselector) clone_app $1 "https://github.com/orgs/nextcloud/globalsiteselector" ;;
music) clone_app $1 "https://github.com/nc-music/music" ;;
onlyoffice) clone_app $1 "https://github.com/ONLYOFFICE/onlyoffice-nextcloud" ;;
*)
cat << EOF
The '$1' app is not recognized by this script. If it is not a typo please open an issue to include it.
Valid extra apps to install:
music
onlyoffice
EOF
exit 1
;;
esac
}
...
for app in "${APPS_TO_INSTALL[@]}"
do
install_app "$app"
done
- Lingua principale
- PHP
- Stelle
- 190
- Fork
- 103
- Merge medio
- 4h 36m
- PR unite (30g)
- 1
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di nextcloud/nextcloud-docker-dev
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
nextcloud/nextcloud-docker-dev#483 · 1 commento · 3 reazioni ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
nextcloud/nextcloud-docker-dev#480 · 1 commento ·
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
-
Linux Manjaro support Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
nextcloud/nextcloud-docker-dev#446 · 1 commento · 1 reazione ·
Tutte le issue di nextcloud/nextcloud-docker-dev
Issue simili
-
priority: p3
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
googleapis/librarian#7636 ·
-
0. Needs triage bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
nextcloud/fulltextsearch#1011 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
phpstan/phpstan-doctrine#794 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
Automattic/static-site-importer#1767 ·