Web: Townsquare feed should recognize links

Created on 25 Mar 2020  ·  27Comments  ·  Source: gitcoinco/web

What

Townsquare feed should recognize links - make link clickable

Why

not clickable

Screen Shot 2020-03-25 at 8 51 18 AM

For Contributor easy-pickings good first issue

All 27 comments

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


__This issue now has a funding of 0.25 ETH (33.53 USD @ $134.12/ETH) attached to it.__

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


__Work has been started__.

These users each claimed they can complete the work by 4 days, 7 hours from now.
Please review their action plans below:

1) sebastiantf has been approved to start work.

Write code to detect links in townsquare and interpret them as hyperlinks

Learn more on the Gitcoin Issue Details page.

@sebastiantf ^ looking forward to seeing a pr!

Yes @PixelantDesign . I'm working on it.

@PixelantDesign I am not able to reproduce the issue rn. It seems the posts on Townsquare do recognize links and are clickable.

Can you please elaborate on the issue about when and how it happens?

it looks like this is a post that were made to a grant wall.

@molecula451 You mean the Activity tab on Grants page? But there too links are being recognized and clickable

Screenshot 2020-03-27 at 12 30 33 PM

@walidmujahid

I have seen the link issue resolved. I am not sure who fixed it or what commit did so, but links do work now for the wall_post activities.

I am not working on this issue, but if this issue still has problems, I am just going to copy and paste a reply I posted the other day in TownSquare that seems relevant - Edited to reflect the lines seen a few days ago:

From: https://gitcoin.co/townsquare?tab=activity:186156

@amiyatulu All wall_post activities suffer from this actually, not just those posted from Grant walls. It is actually not just the links that are an issue -which is done with the urlify function in the activity.js script-, but metadat in general does not show up, nor does the new video call feature.

It feels like only status_update activities are getting handled by any relevant css and the activity.js script, so the solution is probably to make sure that the script applies to wall_post activities as well.

Your main two files to look at for activities, in general, are the retail shared/activities.html -there are multiple shared/*.html templates and it can get very annoying for me even after staring at both code and markup/Gitcoin's codebase for a few months now.

Specifically, you want to look at these lines in the template markup: https://github.com/gitcoinco/web/blob/82823702e609dc289ef3d59b44c559f2a4685498/app/retail/templates/shared/activity.html#L197-L213

This javascript file: https://github.com/gitcoinco/web/blob/master/app/assets/v2/js/activity.js

And a large chunk of the retail.views starting from the beginning -you may just want to read the whole thing, it can help greatly: https://github.com/gitcoinco/web/blob/master/app/retail/views.py

Oh, and here is the model and post_save signal function handler: https://github.com/gitcoinco/web/blob/82823702e609dc289ef3d59b44c559f2a4685498/app/dashboard/models.py#L2047-L2369

@sebastiantf Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • [x] reminder (3 days)
  • [ ] escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

Yes @gitcoinbot.

@PixelantDesign Can you please clarify whether this issue still exists?

@sebastiantf Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • [x] reminder (3 days)
  • [ ] escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@sebastiantf Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • [x] reminder (3 days)
  • [ ] escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

Yes @gitcoinbot

@PixelantDesign Can you please verify whether this issue still exist?

@sebastiantf Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • [x] reminder (3 days)
  • [ ] escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

Yes @gitcoinbot

@PixelantDesign Can you please verify whether this issue still exist?

@sebastiantf
this issue is still open, today I saw some links with this problem.

@developerfred Thanks for notifying.

Will you be able to share a link to that post you saw as not working. Or maybe a screenshot? I am not able to reproduce this issue. All links I saw on the Grants activity wall are recognized and clickable.

@sebastiant I used the Trust Wallet browser, I'm going to record a screemshoot

@developerfred What do you mean the pull request branch?

@developerfred What do you mean the pull request branch?

Sorry, pull request is about another issue

Thanks for sharing the screenshot and link @developerfred. I could reproduce the issue in Chrome too.

But I do think this has something to do with how the user has typed in / entered the url in this case which looks a bit unconventional.

Still the link is being identified and preview is being shown.

But the link is not clickable in the post. It must have something to do with how links are identified and replaced with anchor elements.

🙏🏻 Thank you for identifying this and reporting this. Appreciate this. I can now look into this issue.

@sebastiantf Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • [x] reminder (3 days)
  • [ ] escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

Yea @gitcoinbot

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


__Work for 0.25 ETH (42.98 USD @ $171.91/ETH) has been submitted by__:

  1. @sebastiantf

@PixelantDesign please take a look at the submitted work:

  • PR by @sebastiantf

⚡️ A tip worth 0.03750 ETH (8.39 USD @ $223.65/ETH) has been granted to @sebastiantf for this issue from @PixelantDesign. ⚡️

Nice work @sebastiantf! Your tip has automatically been deposited in the ETH address we have on file.

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


__The funding of 0.25 ETH (55.91 USD @ $223.65/ETH) attached to this issue has been approved & issued to @sebastiantf.__

Additional Tips for this Bounty:

  • PixelantDesign tipped 0.0375 ETH worth 8.39 USD to sebastiantf.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Skyge picture Skyge  ·  3Comments

owocki picture owocki  ·  4Comments

wizzfile picture wizzfile  ·  3Comments

thelostone-mc picture thelostone-mc  ·  4Comments

frankchen07 picture frankchen07  ·  4Comments