DGNum/gestioCOF
13
2
Fork 1
Code Issues 77 Pull requests 14 Packages Projects Releases 12 Wiki Activity

URLs absolues pour sitecof #753

Closed
mpepin wants to merge 1 commit from kerl/sitecof_absolute_urls into master
pull from: kerl/sitecof_absolute_urls
merge into: DGNum:master
Conversation 7 Commits 1 Files changed 1 +12 -6
mpepin commented 2020-08-30 15:18:59 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

(Solution alternative à !448)

On sert toutes les pages du site du COF avec des URLs absolues, ainsi Daphne ne les réécrit pas.

Reste à faire en production :

  • changer le slug du site de 'sitecof' à 'news'
  • mettre en place un reverse-proxy '/news' → 'Daphne:/news'

Avec ça on a de belles URLs pour le site du COF sans trop de hacks.

(Solution alternative à !448) On sert toutes les pages du site du COF avec des URLs absolues, ainsi Daphne ne les réécrit pas. Reste à faire en production : - changer le slug du site de 'sitecof' à 'news' - mettre en place un reverse-proxy '/news' → 'Daphne:/news' Avec ça on a de belles URLs pour le site du COF sans trop de hacks.
mpepin commented 2020-08-30 15:19:28 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

marked as a Work In Progress

marked as a **Work In Progress**
mpepin commented 2020-08-30 15:19:28 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed the description

changed the description
mpepin commented 2020-08-30 15:21:05 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • e56123e5 - Sitecof: always use absolute URLs

Compare with previous version

added 1 commit <ul><li>e56123e5 - Sitecof: always use absolute URLs</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/merge_requests/449/diffs?diff_id=2051&start_sha=374961b6bcfd5aefba48c60789eab8e0749d60e8)
mpepin commented 2020-08-30 15:21:21 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

unmarked as a Work In Progress

unmarked as a **Work In Progress**
champeno commented 2020-08-30 16:41:21 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

il faut aussi redéfinir la propriété url sinon elle fait référence à Page.get_url() et les endroits où url est utilisée (p.ex dans la résolution des liens des RichTextBlock des StreamField) ne fonctionnent pas.

    url = property(get_url)
il faut aussi redéfinir la propriété `url` sinon elle fait référence à `Page.get_url()` et les endroits où `url` est utilisée (p.ex dans la résolution des liens des `RichTextBlock` des `StreamField`) ne fonctionnent pas. ```suggestion:-0+0 url = property(get_url) ```
mpepin commented 2020-08-30 18:57:19 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in merge request !450

mentioned in merge request !450
mpepin commented 2020-08-30 19:05:11 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

closed

closed

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
devtype -- backend

Au programme : du Python, du Django...

  
devtype -- docs

   
devtype -- frontend

   
devtype -- user interface

Au programme : templating, JS, CSS...

  
difficulty -- easy

   
difficulty -- hard

Difficile et/ou risqué

  
difficulty -- normal

   
Doing

   
domain -- bda

Concerne l'application BdA

  
domain -- bds

   
domain -- cof

Concerne l'application COF

  
domain -- core

Dépendances externes, CI, linting...

  
domain -- kfet

Concerne l'application K-Fêt

  
Good first issue

   
priority -- high

Urgent quoi

  
priority -- low

   
priority -- medium

   
priority -- staff-wanted

L'équipe dirigeante du domaine en question souhaite cette fonctionnalité.

  
status -- development

En cours de développement

  
status -- discussion

Avant de coder, c'est bien de discuter

  
status -- need review

En attente de code review

  
status -- production

L'ajout/la modification est présente en production

  
status -- ready to merge

Plus qu'à merger !

  
status -- todo

Le développement devrait bientôt commencer

  
To Do

   
type -- bug

Signalement ou correction de bug

  
type -- hygiene

Tout ce qui amène à du code plus propre et un repo mieux organisé.

  
type -- improvement

Ajout/Demande d'évolution mineure d'une fonctionnalité existante

  
type -- new feature

Ajout/Demande d'une nouvelle fonctionnalité

No labels
devtype -- backend
devtype -- docs
devtype -- frontend
devtype -- user interface
difficulty -- easy
difficulty -- hard
difficulty -- normal
Doing
domain -- bda
domain -- bds
domain -- cof
domain -- core
domain -- kfet
Good first issue
priority -- high
priority -- low
priority -- medium
priority -- staff-wanted
status -- development
status -- discussion
status -- need review
status -- production
status -- ready to merge
status -- todo
To Do
type -- bug
type -- hygiene
type -- improvement
type -- new feature
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/gestioCOF#753
No description provided.
Delete branch "kerl/sitecof_absolute_urls"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?