opensavannah / app.drainsavannah.org

A web app that allows citizens to "adopt" a storm drain in Savannah, Ga.
https://drainsavannah.org
BSD 3-Clause "New" or "Revised" License
1 stars 4 forks source link

List of adopted drains not showing as links #26

Closed roberth05 closed 4 years ago

roberth05 commented 5 years ago

After logging in, the list of drains adopted by the current user are not shown as links.

brandonmichaelhunter commented 5 years ago

Is this still a valid issue?

roberth05 commented 5 years ago

I still experience it. Can you check on your machine and verify?

On Wed, Oct 2, 2019, 8:11 PM Brandon Michael Hunter < notifications@github.com> wrote:

Is this still a valid issue?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/opensavannah/app.drainsavannah.org/issues/26?email_source=notifications&email_token=AL5MSRIA6O6A5QXGLTVKRJTQMU2B3A5CNFSM4IVBQHKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAGSEDY#issuecomment-537731599, or mute the thread https://github.com/notifications/unsubscribe-auth/AL5MSROOHQ4AEZKCL6H65MTQMU2B3ANCNFSM4IVBQHKA .

brandonmichaelhunter commented 5 years ago

Yea, its happening on my machine. Where in the code does it associated the user with a drain?

roberth05 commented 5 years ago

The links are generated in _user_things.html.haml.

If you look at the structure in Chrome dev tools, the link is created but the words are outside the tag.

On Wed, Oct 2, 2019, 8:41 PM Brandon Michael Hunter < notifications@github.com> wrote:

Yea, its happening on my machine. Where in the code does it associated the user with a drain?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/opensavannah/app.drainsavannah.org/issues/26?email_source=notifications&email_token=AL5MSRI3GCUKG2NMKTQ26ZDQMU5SJA5CNFSM4IVBQHKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAGTYYY#issuecomment-537738339, or mute the thread https://github.com/notifications/unsubscribe-auth/AL5MSRJFUQNBCUWPPMAW52TQMU5SJANCNFSM4IVBQHKA .

brandonmichaelhunter commented 5 years ago

Ok I can take a look into this...

roberth05 commented 5 years ago

Thank you!

On Wed, Oct 2, 2019, 8:51 PM Brandon Michael Hunter < notifications@github.com> wrote:

Ok I can take a look into this...

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/opensavannah/app.drainsavannah.org/issues/26?email_source=notifications&email_token=AL5MSRP6DZ3VUQJGFQ3TZGDQMU6ZHA5CNFSM4IVBQHKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAGUH3A#issuecomment-537740268, or mute the thread https://github.com/notifications/unsubscribe-auth/AL5MSRN5HKN45BFU5JXAMQTQMU6ZHANCNFSM4IVBQHKA .

brandonmichaelhunter commented 5 years ago

Hey guys, it appears I got this fixed. I'm going to be doing some more testing morning and once completed I'll have this checked in ready for review.

roberth05 commented 5 years ago

Didn't resolve in live version after merge.

roberth05 commented 4 years ago

Was a problem with the records in the DB. Closing