#981 [Pocket] feat: puce "Je viens de raccrocher" sur les fiches - #982
Open
evarisk-kilyan wants to merge 1 commit into
Open
evarisk-kilyan wants to merge 1 commit into
evarisk-kilyan wants to merge 1 commit into
Conversation
… the last recording The chip sits beside the title of the banner on the objects the Pocket link configuration covers. Clicking it asks Pocket for its last recording, mirrors it as a PocketRecording when it is not already, attaches it to the card and reloads the page without the action, so a refresh never replays the import. The banner parameters are handed over by reference, so the chip is appended to the ref block rather than to resprints, which the call list widgets overwrite. Native cards do not serve the bundle of the module: addHtmlHeader adds the stylesheet and the script only on the cards that actually carry the chip. PocketSync keeps refreshing a recording already mirrored whatever folder it sits in: the chip imports recordings before Pocket files them, and dropping them on the folder filter would freeze them without transcript nor summary.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Close #981
Ce que ça fait
Une puce « Je viens de raccrocher » à côté du titre de la bannière, sur les objets couverts par la configuration des liens Pocket. Un clic demande à Pocket son dernier enregistrement, le met en miroir en
PocketRecordings'il ne l'est pas déjà, le rattache à la fiche, puis recharge la page sans l'action dans l'URL pour qu'un rafraîchissement ne rejoue pas l'import.Les cinq cas de l'issue sont couverts : rattaché, déjà rattaché, enregistrement ancien (avertissement daté), aucun enregistrement, import en échec.
Points d'implémentation
formDolBannerreçoit ses paramètres par référence, la puce est donc ajoutée àmorehtmlrefet non àresprints, que les widgets de liste d'appel écrasent plus bas dans le même hook.addHtmlHeaderajoute la feuille de style etpocket_hangup.jsuniquement sur les fiches qui portent réellement la puce — la liste suit la configuration, elle n'est pas écrite en dur.PocketSync: un enregistrement déjà en miroir continue d'être rafraîchi quel que soit le dossier où il se trouve. La puce importe des enregistrements avant que Pocket ne les classe ; les écarter sur le filtre de dossier les figerait sans transcription ni résumé. Chaque enregistrement est estampillé du dossier où il est réellement.Vérifications
php -lsur les 4 fichiers PHP modifiés : aucune erreur.build-assets.js, sass + esbuild). Contrôle de déterminisme fait au préalable : rebuild d'unorigin/developvierge dans un worktree détaché,git statusvide — la chaîne locale reproduit bien celle du CI.Contexte
Ce travail était resté non commité dans un arbre de travail et n'existait sur aucune branche. Il est remis ici tel quel, à deux exceptions près qui ne relèvent pas de cette PR :
develop, n'est pas repris ;js/modules/inline_typeahead.js, non suivi lui aussi, a été écarté du bundle pour qu'il n'entre pas dansdeveloppar la bande.