search

cthit / chalmersit-rails

Chalmers.it built with Rails
MIT License
4 stars 6 forks source link

Use image.url #399

Closed Kalior closed 5 years ago

Kalior commented 5 years ago

For some reason the image_tag function seems to work differently on html.erb files and in the helper functions?

NeonNeon commented 5 years ago

Känns ju väldigt konstigt att det funkar annorlunda i det här fallet. Man använder ju image_tag på samma sätt i andra vyer utan att det genererar ett fel? Jag funderar på om vi istället ska ta bort helper-funktionen, och ha den lilla lilla dupliceringen i _sponsor.html.erb istället. Då är vi ju i alla fall konsekventa över kodbasen (förstår ju fortfarande inte hur detta kan bli olika)

NeonNeon commented 5 years ago

Om man tittar här: https://apidock.com/rails/ActionView/Helpers/AssetTagHelper/image_tag Så står det att man kan skicka in "Active Storage (images that are uploaded by the users of your app)" vilket väl vår sponsor.image är?


image_tag(user.avatar)
=> <img src="/rails/active_storage/blobs/.../tiger.jpg" />```
Kalior commented 5 years ago

Håller med, väldigt konstigt att det är olika, och dokumentationen verkar ju stödja att det ska funka ja. Jag är okej med att flytta det ifrån helper funktionen också.

NeonNeon commented 5 years ago

Hur har du testat ifall detta funkar btw? Testar du i produktion?

Kalior commented 5 years ago

Yes, testade det på servern innan jag la upp detta. Jag antar att det redan funkar lokalt?

NeonNeon commented 5 years ago

Okej, men jag förstod i slack att det fortfarande var saker som var trasiga? Tänker att det är dumt att mergea in något innan vi är säkra på att vi rättar rätt sak.

Det känns som att vi försöker dämpa symptom på olika fronter medans det är egentligen är något fundamentalt som har blivit fel och borde rättas upp.

Gurrit commented 5 years ago

Allting ju verkar funka i prod, men som sagt känns det lite oklart varför. Tycker att det borde mergas in, men att vi borde kolla på om det var API:t som ändrades eller något annat hände. Jag gör ett issue på det om ingen är emot

NeonNeon commented 5 years ago

Allting ju verkar funka i prod, men som sagt känns det lite oklart varför. Tycker att det borde mergas in, men att vi borde kolla på om det var API:t som ändrades eller något annat hände. Jag gör ett issue på det om ingen är emot

Låter bra