search

jesuisundesdeux / vigilo-webapp

GNU General Public License v3.0
14 stars 10 forks source link

pb affichage photo #66

Closed ncoustenoble closed 3 years ago

ncoustenoble commented 3 years ago

Bug

je souhaiterais modifier une photo sur mon signalement AF36D88C

Navigateur

Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:82.0) Gecko/20100101 Firefox/82.0

Zone géographique

Aix Marseille Provence Metropole

Version

tcoupin commented 3 years ago

@kentiss34 je stocke la clé privée liée au signalement dans le navigateur. Je pourrais donc permettre l'édition par la personne qui a envoyé le signalement (avec nouvelle modération donc...)

qhess34 commented 3 years ago

Le soucis par rapport à ça, c'est le risque de changement complet de l'observation qui serait donc différente de ce qui a été posté sur twitter ou entré dans d'autres outils créant des incohérences vis à vis du token et de l'id qui eux ne bougent pas.

On peut envisager une option consistant à supprimer l'obs et en recréer une (donc avec nouveau token et id) avec un formulaire d'édition sur lequel les champs sont pré-remplis avec le contenu de l'observation à modifier.

Qu'en penses tu ?

Le dim. 22 nov. 2020 à 17:32, Thibault Coupin notifications@github.com a écrit :

@kentiss34 https://github.com/kentiss34 je stocke la clé privée liée au signalement dans le navigateur. Je pourrais donc permettre l'édition par la personne qui a envoyé le signalement (avec nouvelle modération donc...)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jesuisundesdeux/vigilo-webapp/issues/66#issuecomment-731773492, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACBMIQPVDXZW2XZTBQVUHALSRE4IDANCNFSM4TW26DSA .

tcoupin commented 3 years ago

La suppression d'une obs supprime le tweet associé ?

qhess34 commented 3 years ago

Non, la seule interaction avec Twitter se limite à la publication des observations à leur création.

Le dim. 29 nov. 2020 à 20:59, Thibault Coupin notifications@github.com a écrit :

La suppression d'une obs supprime le tweet associé ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jesuisundesdeux/vigilo-webapp/issues/66#issuecomment-735445671, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACBMIQIRTMCWTWW6LFSIUPTSSKR2LANCNFSM4TW26DSA .

tcoupin commented 3 years ago

donc la suppression de l'obs pour en recréer une autre engendrerait une situation tout aussi inconfortable avec un tweet incohérent puisque pointant sur une obs non existante

qhess34 commented 3 years ago

effectivement ! Les observations sont tweetées au moment où elles sont approuvées en réalité, on pourrait donc partir sur la désactivation de l'option de suppression quand l'obs est activée.

On Sun, 29 Nov 2020 at 21:23, Thibault Coupin notifications@github.com wrote:

donc la suppression de l'obs pour en recréer une autre engendrerait une situation tout aussi inconfortable avec un tweet incohérent puisque pointant sur une obs non existante

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jesuisundesdeux/vigilo-webapp/issues/66#issuecomment-735448461, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACBMIQOBAJT4QCQHP2VSJ3LSSKUT5ANCNFSM4TW26DSA .

tcoupin commented 3 years ago

Bien vu !