forked from codefordurham/adopt-a-drain
-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
List of adopted drains not showing as links #26
Labels
bug
Something isn't working
Comments
Is this still a valid issue? |
I still experience it. Can you check on your machine and verify?
…On Wed, Oct 2, 2019, 8:11 PM Brandon Michael Hunter < ***@***.***> 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
<#26?email_source=notifications&email_token=AL5MSRIA6O6A5QXGLTVKRJTQMU2B3A5CNFSM4IVBQHKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAGSEDY#issuecomment-537731599>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AL5MSROOHQ4AEZKCL6H65MTQMU2B3ANCNFSM4IVBQHKA>
.
|
Yea, its happening on my machine. Where in the code does it associated the user with a drain? |
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 < ***@***.***> 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
<#26?email_source=notifications&email_token=AL5MSRI3GCUKG2NMKTQ26ZDQMU5SJA5CNFSM4IVBQHKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAGTYYY#issuecomment-537738339>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AL5MSRJFUQNBCUWPPMAW52TQMU5SJANCNFSM4IVBQHKA>
.
|
Ok I can take a look into this... |
Thank you!
…On Wed, Oct 2, 2019, 8:51 PM Brandon Michael Hunter < ***@***.***> 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
<#26?email_source=notifications&email_token=AL5MSRP6DZ3VUQJGFQ3TZGDQMU6ZHA5CNFSM4IVBQHKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAGUH3A#issuecomment-537740268>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AL5MSRN5HKN45BFU5JXAMQTQMU6ZHANCNFSM4IVBQHKA>
.
|
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. |
brandonmichaelhunter
pushed a commit
that referenced
this issue
Oct 12, 2019
Merged
roberth05
added a commit
that referenced
this issue
Nov 2, 2019
Didn't resolve in live version after merge. |
Was a problem with the records in the DB. Closing |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
After logging in, the list of drains adopted by the current user are not shown as links.
The text was updated successfully, but these errors were encountered: