search

DispuutHamers / webapp

https://www.zondersikkel.nl/
6 stars 0 forks source link

You cannot add your own chugs #482

Closed koenjdejong closed 2 years ago

koenjdejong commented 2 years ago

Closes #480

dexbleeker commented 2 years ago

Very good idea! Maybe also do this for quotes?

koenjdejong commented 2 years ago

Je kan dit volgens mij beter oplossen met een validation op de model. Dan is er ook nooit ergens anders waar het wel mis kan gaan.

Probleem waar ik dan mee zit is hoe ik de current_user krijg in chug.rb, waar ik geen reporter heb zoals in quote. Er wordt namelijk nog niks meegestuurd als reporter.

jackozi commented 2 years ago

Je kan dit volgens mij beter oplossen met een validation op de model. Dan is er ook nooit ergens anders waar het wel mis kan gaan.

Probleem waar ik dan mee zit is hoe ik de current_user krijg in chug.rb, waar ik geen reporter heb zoals in quote. Er wordt namelijk nog niks meegestuurd als reporter.

Wat je zou kunnen doen is een attr_reader op de model zetten en dan meesturen via een hidden field, of wel de reporter gaan bijhouden zoals in quotes. Is misschien later ook handig voor auditing purposes als iemand een net iets te snelle tijd neerzet.

koenjdejong commented 2 years ago

Je kan dit volgens mij beter oplossen met een validation op de model. Dan is er ook nooit ergens anders waar het wel mis kan gaan.

Probleem waar ik dan mee zit is hoe ik de current_user krijg in chug.rb, waar ik geen reporter heb zoals in quote. Er wordt namelijk nog niks meegestuurd als reporter.

Wat je zou kunnen doen is een attr_reader op de model zetten en dan meesturen via een hidden field, of wel de reporter gaan bijhouden zoals in quotes. Is misschien later ook handig voor auditing purposes als iemand een net iets te snelle tijd neerzet.

Reporter bijhouden is inderdaad een goed idee, kijk ik later even naar