Corrections de la journalisation des éditions d'objets #4915

Open
MathieuAlphamosa wants to merge 2 commits from MathieuAlphamosa/spip:issue_4913 into master

fix #4913

  • fix : id et nom de l'auteur
  • fix : caractères encodés et suppression du "L apostrophe"
fix #4913 - fix : id et nom de l'auteur - fix : caractères encodés et suppression du "L apostrophe"
MathieuAlphamosa added 1 commit 1 week ago
f47868c8e7 Corrections de la journalisation des éditions d'objets
b_b commented 1 week ago
Owner

Bon pour moi, sauf que tu retires une info avec ta modif : avant on loguait l'ip si on n'avait pas de session.

Bon pour moi, sauf que tu retires une info avec ta modif : avant on loguait l'ip si on n'avait pas de session.
MathieuAlphamosa added 1 commit 1 week ago

@b_b : Bien vu, c'est corrigé.

@b_b : Bien vu, c'est corrigé.
b_b commented 3 days ago
Owner

Ha mince, ya un conflit, tu peux le résoudre de ton côté @MathieuAlphamosa ? Et tant qu'à faire, un ty rebase -i pour unifier tout ça en un seul commit :)

Ha mince, ya un conflit, tu peux le résoudre de ton côté @MathieuAlphamosa ? Et tant qu'à faire, un ty rebase -i pour unifier tout ça en un seul commit :)
b_b added the
bug
label 3 days ago
b_b added this to the 4.0 milestone 3 days ago

Je crois que je vais arrêter de proposer des modifications tant que j'ai pas fait une bonne formation git, car pour l'instant je me contente de l'interface web, mais là impossible de faire ce que tu me demandes @b_b, désolé :(

Je crois que je vais arrêter de proposer des modifications tant que j'ai pas fait une bonne formation git, car pour l'instant je me contente de l'interface web, mais là impossible de faire ce que tu me demandes @b_b, désolé :(
b_b commented 3 days ago
Owner

Pas de problème @MathieuAlphamosa on peut faire simple aussi : tu récupère tes modifications ici, tu fermes la PR et tu en crées une nouvelle, et zou :)

Pas de problème @MathieuAlphamosa on peut faire simple aussi : tu récupère tes modifications ici, tu fermes la PR et tu en crées une nouvelle, et zou :)
This pull request has changes conflicting with the target branch.
ecrire/inc/modifier.php
Sign in to join this conversation.
Loading…
There is no content yet.