Support preview link surround by text - Githubissues

NattapongSiri / covid_cb

MIT License

2 stars 1 forks source link

Support preview link surround by text #6

Closed NattapongSiri closed 4 years ago

NattapongSiri commented 4 years ago

By current design, it'll require a link to be the first element in the text. It'd be more useful to be able to preview a link that is surround by other text as well.

Some questions needed to be answer is:

What if there's more than one link ?
- Show only first link ?
- Show only last link ?
- Show all the links ?
  - As single large message from server ?
  - Each individual preview is inside it own message ?
Shall we also support a link that's not in
tag ?
- How can we know the boundary of link in this case ? Space terminated ?

Potential code area to be affected:

NattapongSiri commented 4 years ago

End up fix by using regex to detect all possible http(s)://url patterns then create each message with previewUrl for each extracted pattern.