Skip to content
Snippets Groups Projects

Updates

Merged Baptiste GAUDRON requested to merge updates into master
1 unresolved thread
  • Add possibility to plot profiles selecting colormap style or normal plot
  • Store selected values with SessionStorage to keep it after refreshing the page

Merge request reports

Merged by sylvie fiatsylvie fiat 2 years ago (Feb 2, 2023 4:43am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
44 function setWithExpiry(key, value, ttl) {
45 const now = new Date();
46
47 // `item` is an object which contains the original value
48 // as well as the time when it's supposed to expire
49 const item = {
50 value: value,
51 expiry: now.getTime() + ttl,
52 };
53 sessionStorage.setItem(key, JSON.stringify(item));
54 }
55
56 function keepSelectValue(selectElem) {
57 setWithExpiry(jQuery(selectElem).attr('id'), jQuery(selectElem).val(), expiryTimeInterval);
58 }
59
  • Je pense que mettre une date d'expiration comme ça (ie qui supprime la valeur - si périmée - quand la personne se reconnecte) dans notre cas (on ne stocke pas des droits d'accès mais une valeur de choix dans un formulaire...) ça ne sert à rien... je t'ai demandé de mettre une date de péremption pour la loi rgpd et éviter (par principe) de stocker des infos dans les navigateurs des gens... mais d'après ce que je lis sur la cnil (https://www.cnil.fr/en/node/120386) notre cas rentre dans les traceurs exemptés de consentement, je vais donc juste vérifier que notre page de mentions légales est ok sur ce point et tu me supprime l'Expirity de ce code s'il te plait :)

  • Please register or sign in to reply
  • c'est ok pour le reste je valide dès que tu m'auras enlevé l'expiration

  • added 1 commit

    • 508c352b - remove expiry time for saved options

    Compare with previous version

  • C'est fait !

  • sylvie fiat approved this merge request

    approved this merge request

  • merged

  • sylvie fiat mentioned in commit 88b4f239

    mentioned in commit 88b4f239

  • Please register or sign in to reply
    Loading