Projet

Général

Profil

Development #9632

MandayeJS : arpege app settings

Ajouté par Josué Kouka il y a plus de 8 ans. Mis à jour il y a environ 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Josué Kouka
Catégorie:
-
Version cible:
-
Début:
13 janvier 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Fichiers


Demandes liées

Lié à Mandaye - Development #9732: MandayeJS : Gérer l'inclusion de jQuery quand l'application ne la charge pas par defautFermé21 janvier 2016

Actions
Lié à Mandaye - Development #9727: Permette à un AppSettings de fournirs des URLsFermé21 janvier 2016

Actions

Révisions associées

Révision 58947660 (diff)
Ajouté par Josué Kouka il y a plus de 8 ans

add arpege app settings (#9632)

Historique

#1

Mis à jour par Josué Kouka il y a plus de 8 ans

  • Statut changé de Nouveau à En cours

En me basant sur l'espace famille de Vincennes https://vincennes-test.espace-famille.net/vincennes-test/index.do

#2

Mis à jour par Josué Kouka il y a plus de 8 ans

semble ne pas avoir jquery par defaut. Dans la config nginx, ajouter :

sub_filter </head> '<script type="text/javascript" src="$scheme://$server_name/_mandaye/static/xstatic/jquery.min.js"></script> <script type="text/javascript" src="$scheme://$server_name/_mandaye/static/mandaye.js"></script></head>';

#4

Mis à jour par Josué Kouka il y a plus de 8 ans

  • Patch proposed changé de Non à Oui
#5

Mis à jour par Frédéric Péters il y a plus de 8 ans

On pourrait imaginer que notre mandaye.js fonctionne sans jquery et commence par ajouter le <script src="...jquery.js"> si jquery n'est pas trouvé ? (plutôt que d'avoir une variété de configurations nginx à entretenir)

#6

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

Oui soit l'un soit l'autre:
  • ne pas utiliser du tout jQuery (peut être un peu chiant)
  • renommer jQuery pour notre usage en mandaye_jQuery, et encadrer tous les codes par
    (function ($) {
     ....
    })(mandaye_jQuery)
    

Il faut vraiment éviter de polluer les espaces de nom JS (ça vaut aussi pour les classes CSS, etc..)

#7

Mis à jour par Josué Kouka il y a plus de 8 ans

Yep pas faux, je l'ajoute dans le mandaye.js ( je pense avoir un problème avec la vision "globale" des choses lol).

#8

Mis à jour par Josué Kouka il y a plus de 8 ans

  • Lié à Development #9732: MandayeJS : Gérer l'inclusion de jQuery quand l'application ne la charge pas par defaut ajouté
#10

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

Pas besoin de créer une nouvelle application juste pour un fichier JS, met le à la racine (c'est justement ce que je voulais éviter, avoir pleins d'applications pour rien).

#11

Mis à jour par Josué Kouka il y a plus de 8 ans

Le statics 'auth.checker.js' est relatif à chaque applications. Du coup pour le versionning je suis obligé de le placer comme tel. Ensuite, en fonction de l'app par tenant, je pourrai le placer à la racine si besoin. C'est ce que je fais pour les statics par client.

#12

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

Je ne comprends pas. Le script se nomme 'arpege/js/authchecker.js' il est parfaitement séparé des autres scripts, pourquoi avoir besoin d'une application Django ?

#13

Mis à jour par Josué Kouka il y a plus de 8 ans

On était parti sur l'idée que chaque type d'application "mandayisée" est une "app" django ou du moins un package qui y ressemble. L'interet principal était de pouvoir écrire des views pour les api si jamais elles en fournissaient.

#14

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

Il n'y a pas besoin d'une app django pour écrire des vues (c'est tout l'intérêt de mon autre ticket) le seul cas nécessitant une app Django c'est pour ajouter des modèles.

#15

Mis à jour par Josué Kouka il y a plus de 8 ans

#16

Mis à jour par Josué Kouka il y a plus de 8 ans

Ok. Si je te comprends bien.
  • Je vire toutes les apps django correspondant au différentes applications ( duonet, archimed, ..)
  • Je mets leur statics dans mandayejs/mandayejs/static
  • Je mets leur views.py ( pour l'instant seul archimed en a) dans mandayejs/mandayejs/views.py
    ?
#17

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

Tu peux même mettre les views dans les app_settings, ce n'est pas dérangeant:

class Archimed(AppSetting):

    class AccountDetails(APIView):
        ...

    urlpatterns = urlpattern(
        url(r'account/(?P<username>[\w+]*)/$', AccountDetails.as_view()))
#18

Mis à jour par Josué Kouka il y a plus de 8 ans

  • Statut changé de En cours à Résolu (à déployer)
#19

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF