Pc 32405 rpa import book club chronicles #15253
Conversation
rpaoloni-pass
force-pushed
the
pc-32405-rpa-import-book-club-chronicles
branch
from
0d229c8 to
ac5527e
Compare
rpaoloni-pass
force-pushed
the
pc-32405-rpa-import-book-club-chronicles
branch
from
ac5527e to
ac56704
Compare
| @@ -1,20 +1,20 @@ | |||
| {% macro collapsable_text(id, short, long) %} | |||
| <div class="pc-display-selector" | |||
| data-pc-input-name="name-{{ id }}"> | |||
There was a problem hiding this comment.
Petit screenshot ?
Je suis une personne simple, je vois des modifs de HTML, je demande un screenshot 😉
There was a problem hiding this comment.
ça ne change rien a l'affichage ça résout juste un bug dans le js (le texte qui est étendu par défaut au lieu d'être coupé par défaut)
| @@ -53,3 +53,24 @@ def wrapper(*args: typing.Any, **kwargs: typing.Any) -> typing.Any: | |||
| return result | |||
|
|
|||
| return wrapper | |||
|
|
|||
|
|
|||
| def log_cron(func: typing.Callable) -> typing.Callable: | |||
There was a problem hiding this comment.
Pourquoi on utilise pas log_cron_with_transaction ici ?
There was a problem hiding this comment.
parce que cette fonction est fondamentalement incompatible avec @atomic que j'utilise plus loin
(en gros si on met du atomic dans un log_cron_with_transaction il ets forcement rollback a la fin
| -- Anonymize redactor for table chronicle | ||
| UPDATE "chronicle" | ||
| SET | ||
| "email" = 'chronicle' || "id" || '@anonymized.email', | ||
| "firstName" = pg_temp.fake_first_name(id); |
| form_id=constants.BOOK_CLUB_FORM_ID, | ||
| num_results=constants.IMPORT_CHUNK_SIZE, | ||
| sort="submitted_at,asc", | ||
| since=last_chronicle.dateCreated if last_chronicle else None, |
There was a problem hiding this comment.
Point de vigilance : attention, cela signifie qu'on ne doit pas ajouter de nouvelles chroniques en dehors de ce chemin, car on ignorerait alors toutes celles ajoutées sur Typeform et précédant la date d'une chronique ajoutée par cet autre biais.
There was a problem hiding this comment.
en effet mais je ne vois pas trop comment faire autrement. L'argument after de typeform ne supporte pas l'argument sort (et du coup il ne sert a rien)
| if($selector.dataset.pcSaveState === "true") { | ||
| const selectedValue = localStorage.getItem($selector.dataset.pcInputName) | ||
| this.#inputInSelector($selector).forEach(($input) => { | ||
| if ($input.value === selectedValue){ | ||
| $input.checked = true | ||
| } | ||
| }) | ||
| } else { | ||
| localStorage.removeItem($selector.dataset.pcInputName) | ||
| } | ||
| } | ||
| #saveConfiguration = ($selector) => { | ||
| localStorage.setItem($selector.dataset.pcInputName, this.#getSelectedValue($selector)) | ||
| if($selector.dataset.pcSaveState === "true") { | ||
| localStorage.setItem($selector.dataset.pcInputName, this.#getSelectedValue($selector)) | ||
| } |
There was a problem hiding this comment.
Je ne devine pas dans quelle page est le bug ni si ça a un rapport avec le BookClub.
Quel est le bug ?
There was a problem hiding this comment.
Le bug est sur le repliement du texte sur la page de liste des chroniques. Il retenait les chroniques pliées/dépliées ce que je trouve être un comportement peu intuitif. Les modifications de ce commit font que:
- a chaque fois qu'on arrive sur la page toutes les chroniques sont dans le même état
- que l'etat par defaut soit l'etat plié
There was a problem hiding this comment.
Ce n'est pas forcément le sujet de ce ticket mais... Comment un utilisateur du BO devine-t-il qu'il faut cliquer pour déplier ? Question UX à poser :)
There was a problem hiding this comment.
les points de suspension me paraissaient suffisant mais si ça ne l'est pas pour toi, il faut en effet en parler niveau UX :)
rpaoloni-pass
force-pushed
the
pc-32405-rpa-import-book-club-chronicles
branch
from
ac56704 to
58dc55a
Compare
rpaoloni-pass
force-pushed
the
pc-32405-rpa-import-book-club-chronicles
branch
from
58dc55a to
5f83c2e
Compare
But de la pull request
Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-32405
Vérifications