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

Draft: Authens ! #767

Open
mpepin wants to merge 5 commits from kerl/authens into master
pull from: kerl/authens
merge into: DGNum:master
Conversation 11 Commits 5 Files changed 18 +114 -141
mpepin commented 2020-09-17 21:40:11 +02:00 (Migrated from git.eleves.ens.fr)
Copy link
  • On utilise authens partout (pas que dans GestioBDS)
  • J'ai monkey-patch le CSS d'authens pour que ce soit similaire à GestioCOF, pour moi ça suffit je ne suis pas contre un second avis
  • TODO DONE: migrer les comptes existants vers authens
- On utilise authens partout (pas que dans GestioBDS) - J'ai monkey-patch le CSS d'authens pour que ce soit similaire à GestioCOF, pour moi ça suffit je ne suis pas contre un second avis - **~~TODO~~ DONE**: migrer les comptes existants vers authens
mpepin commented 2020-09-22 20:54:53 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • 6e1e4631 - Migrate django_cas_ng accounts to authens

Compare with previous version

added 1 commit <ul><li>6e1e4631 - Migrate django_cas_ng accounts to authens</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/463/diffs?diff_id=2158&start_sha=8f5b43c010cf28efaebe588e3707060fa91caf7b)
mpepin commented 2020-09-22 20:56:05 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

unmarked as a Work In Progress

unmarked as a **Work In Progress**
mpepin commented 2020-09-22 20:56:05 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed the description

changed the description
mpepin commented 2020-09-22 21:42:50 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 23 commits

  • 6e1e4631...3869c02d - 18 commits from branch master
  • 26455ae5 - Monkey-patch authens to make it look like GestioCOF
  • a98e56d3 - Switch to Authens for GestioCOF authentication
  • 7b8d1575 - Break a redirection loop in teamkfet_required
  • 0815f797 - Update auth urls in tests
  • 3967831e - Migrate django_cas_ng accounts to authens

Compare with previous version

added 23 commits <ul><li>6e1e4631...3869c02d - 18 commits from branch <code>master</code></li><li>26455ae5 - Monkey-patch authens to make it look like GestioCOF</li><li>a98e56d3 - Switch to Authens for GestioCOF authentication</li><li>7b8d1575 - Break a redirection loop in teamkfet_required</li><li>0815f797 - Update auth urls in tests</li><li>3967831e - Migrate django_cas_ng accounts to authens</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/463/diffs?diff_id=2161&start_sha=6e1e46311f731d65d6a2d8e16f509241d0ce8a6f)
lstephan commented 2020-09-25 16:31:01 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ce changement fait raler isort... Il doit y avoir un pb de version.

Ce changement fait raler isort... Il doit y avoir un pb de version.
lstephan commented 2020-09-25 16:32:30 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Si c'est commun, il y a un problème à le mettre sur auth seulement ?

Si c'est commun, il y a un problème à le mettre sur `auth` seulement ?
mpepin commented 2020-09-26 23:33:23 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je pense que ça a plus sa place sous {cof,bds}.ens.fr/gestion/auth que sous {cof,bds}.ens.fr/auth tout court, c'est l'auth de GestioCOF/GestioBDS, pas forcément des autres trucs que peut vouloir mettre sous ces noms de domaine.

Je pense que ça a plus sa place sous `{cof,bds}.ens.fr/gestion/auth` que sous `{cof,bds}.ens.fr/auth` tout court, c'est l'auth de GestioCOF/GestioBDS, pas forcément des autres trucs que peut vouloir mettre sous ces noms de domaine.
mpepin commented 2020-09-26 23:57:02 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Hum, en fait il y a encore un million de trucs à changer dans le code dès lors qu'on touche à CofProfile.login_clipper, va savoir pourquoi c'est utilisé partout

Hum, en fait il y a encore un million de trucs à changer dans le code dès lors qu'on touche à `CofProfile.login_clipper`, va savoir pourquoi c'est utilisé partout
mpepin commented 2021-02-07 19:35:48 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

marked this merge request as draft

marked this merge request as **draft**
mpepin commented 2021-02-07 19:35:49 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

changed the description

changed the description
lstephan commented 2021-03-04 09:21:49 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in issue #288

mentioned in issue #288
This pull request has changes conflicting with the target branch.
  • cof/settings/bds_prod.py
  • cof/settings/cof_prod.py
  • cof/settings/common.py
  • cof/urls.py
  • gestioncof/views.py
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin kerl/authens:kerl/authens
git checkout kerl/authens

Merge

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff kerl/authens
git checkout master
git merge --ff-only kerl/authens
git checkout kerl/authens
git rebase master
git checkout master
git merge --no-ff kerl/authens
git checkout master
git merge --squash kerl/authens
git checkout master
git merge --ff-only kerl/authens
git checkout master
git merge kerl/authens
git push origin master
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#767
No description provided.
Delete branch "kerl/authens"

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?