[bug] Assumes all apps can be cloned from the `nextcloud` org (including suggested fix)
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
Línea de trabajo
Comienza en bootstrap.sh en la línea enlazada alrededor de la 158 y sigue cómo se clonan las aplicaciones adicionales solicitadas. Ejecuta ./bootstrap.sh music antes y después del cambio; se considera terminado cuando la aplicación Music se obtiene de https://github.com/nc-music/music en lugar de asumir que pertenece a la organización nextcloud.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- PHP
- Estrellas
- 190
- Forks
- 103
- Merge medio
- 4 h 36 min
- PR fusionados (30 d)
- 1
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nextcloud/nextcloud-docker-dev
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
nextcloud/nextcloud-docker-dev#483 · 1 comentario · 3 reacciones ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
nextcloud/nextcloud-docker-dev#480 · 1 comentario ·
-
Setup does not work with Selinux Abiertobug
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
-
Linux Manjaro support Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
nextcloud/nextcloud-docker-dev#446 · 1 comentario · 1 reacción ·
Todos los issues de nextcloud/nextcloud-docker-dev
Issues similares
-
priority: p3
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
googleapis/librarian#7636 ·
-
0. Needs triage bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
nextcloud/fulltextsearch#1011 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
coollabsio/coolify#11927 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100