-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(api): #1977 disable inactive for two years #2021
Conversation
486dbdc
to
fcaf397
Compare
Je vais rajouter un test d'intégration, mais on peut déjà relire |
J'ai rajouté le test d'integ, mais je suis pas contente parce qu'il faut refresh le snapshot tous les deux ans parce que je n'ai pas réussi à mocker la date sans que ça fasse des time out. J'ai beaucoup essayé ces solutions-là nock/nock#2200 mais rien à faire. Si vous avez des idées ^^ |
bon en + c'est cassé, je m'y remets cet aprèm' |
Je comprend pas trop mais dispo pour t'aider car j'ai déjà utilisé |
540b390
to
649e305
Compare
Je comprends pas pourquoi le test ne passe pas au CI, en local ça passe sans pb :'( |
35db00b
to
376ea51
Compare
c'est mergeable ! je sais pas s'il vous restait des sujets |
ok pour moi |
re en draft : ça marche pas pour des utilisateurs qui se sont soigneusement déconnectés |
75b585e
to
68ea273
Compare
closes #1977