Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Reminder] Reverse le code de la pagination #2817

Closed
DevHugo opened this issue Jun 17, 2015 · 7 comments
Closed

[Reminder] Reverse le code de la pagination #2817

DevHugo opened this issue Jun 17, 2015 · 7 comments
Labels
C-API Concerne une API du site S-Zombie Ticket ou PR oubliée

Comments

@DevHugo
Copy link
Contributor

DevHugo commented Jun 17, 2015

La correction de la pagination inclue une correction temporaire. Celle-ci devra être modifiée lorsqu'une correction dans la librairie: drf-extension (custom extensions for Django REST Framework) sera proposé dans ce fichier.

Les actions suivante sont à éffectuer:

OU

Il faut donc lire régulièrement les "changelog" ici.

Plus d'infos:
@Eskimon Eskimon added Evolution Facile Bon ticket pour débuter pour rejoindre le développement ! C-API Concerne une API du site labels Jun 18, 2015
@Eskimon Eskimon changed the title Reverse le code de la pagination [Reminder] Reverse le code de la pagination Jun 18, 2015
@vhf
Copy link
Contributor

vhf commented Oct 17, 2016

Ping @DevHugo :p

@vhf vhf removed the Facile Bon ticket pour débuter pour rejoindre le développement ! label Oct 17, 2016
@GerardPaligot
Copy link
Member

GerardPaligot commented Oct 17, 2016 •
edited
Loading

La pagination a été fixée chez drf-extension ?

Je vois ce commit qui est nommé "Fixed PaginationKeyBit" et juste derrière ce commit qui revert le premier commit.

@DevHugo
Copy link
Contributor Author

DevHugo commented Oct 17, 2016

Je fait ça dans la semaine !

DevHugo added a commit to DevHugo/zds-site that referenced this issue Nov 29, 2016
@DevHugo
Reverse le code de la pagination
67d001b 
Pour en savoir plus, voir le ticket zestedesavoir#2817.
@DevHugo DevHugo mentioned this issue Nov 29, 2016
Closed
@DevHugo
Copy link
Contributor Author

DevHugo commented Nov 29, 2016 •
edited
Loading

Ouai, c'est bizarre mais sur la branche de release, c'est le code du premier commit et sur la branche master aussi.

Eh ba non enfait.

Je tente au pire y'a un tu, que j'avais fait qui teste donc normallement pas de souci :) !

Edit: eh ba ça marche pas, faut encore attendre !

@artragis
Copy link
Member

hello on en est où ici?

@artragis artragis added the S-Zombie Ticket ou PR oubliée label Dec 26, 2018
@DevHugo
Copy link
Contributor Author

DevHugo commented Jan 4, 2019 •
edited
Loading

La classe n'a plus l'air d'exister et le comportement marche en prod donc je suppose que quelqu'un a du revert le commit et comme le problème à été résolu dans la librairie, ça fonctionne.

@artragis
Copy link
Member

artragis commented Jan 5, 2019

\o/

@artragis artragis closed this as completed Jan 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-API Concerne une API du site S-Zombie Ticket ou PR oubliée
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@Eskimon @artragis @GerardPaligot @vhf @DevHugo @Situphen