search

code4romania / war-support-un-acoperis

https://unacoperis.ro/ro
Mozilla Public License 2.0
9 stars 37 forks source link

Display other peope in the admin page for reugee requests #161

Closed cstns closed 2 years ago

cstns commented 2 years ago

Requirements for making a pull request

Thank you for contributing to our project!

Please fill out the template below to help the project maintainers review it as fast as possible and include your contribution to the project.

What does it fix?

Closes https://github.com/code4romania/war-support-un-acoperis/issues/143

Please mention the main changes this PR brings.

How has it been tested?

Please describe the tests that you ran to verify your changes.

gheorghelupu17 commented 2 years ago

Salut, dacă înțeleg eu bine e legat de help request, nu înțeleg ce treabă are accommodations. Uite cum înțeleg eu... Aici http://un-acoperis.heroesof.tech/admin/help-request/12 la field other people Screenshot_2022-03-05-03-42-00-636_com android chrome ar trebui să apară ce e in db pentru acel câmp sub forma de listă ori smth, in baza de date acel câmp e un json, deci un haine decode și foreach pe el

beniamin commented 2 years ago

Pare ca PR-ul asta contine si PR-ul cu limitele de la accommodations https://github.com/code4romania/war-support-un-acoperis/pull/154

In plus, problema de afisare acolo nu e de la afisare ci este de la cum se salveaza in DB in formularele initiale.

cstns commented 2 years ago

da, problema era cand se salvau in db, ajungea un json array cu un singur entry cu valorile nule si nu se facea display din cauza asta. Am mai rafinat un pic si eventurile de jquery on pageload/refresh cand se completeaza requestul de ofera cazare.

cstns commented 2 years ago

intradevar, pornisem de pe un alt branch decat development din graba, am facut revert