Skip to content

Fix/repair source link#8

Open
vtexdaniel wants to merge 3 commits into
vtex-apps:masterfrom
vtexdaniel:fix/repair-source-link
Open

Fix/repair source link#8
vtexdaniel wants to merge 3 commits into
vtex-apps:masterfrom
vtexdaniel:fix/repair-source-link

Conversation

@vtexdaniel

Copy link
Copy Markdown

What problem is this solving?

Recently, Tawk.to changed the source link that renders their chat app on the webstore.

Previously, the link was composed of:
'https://embed.tawk.to/' + propertyId + '/default'

Now, it is composed of:
'https://embed.tawk.to/' + propertyId + '/' + linkId

  • where linkId is the string found in the Direct Chat Link

image

How should this be manually tested?
https://danieltest--melimeloparis.myvtex.com/?__bindingAddress=www.melimeloparis.ro/

Screenshots or example usage:
Screenshot 2021-06-26 at 18 01 21

@vtexdaniel vtexdaniel requested review from a team as code owners June 26, 2021 15:21
@vtex-io-ci-cd

vtex-io-ci-cd Bot commented Jun 26, 2021

Copy link
Copy Markdown

Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖

Please select which version do you want to release:

  • Patch (backwards-compatible bug fixes)

  • Minor (backwards-compatible functionality)

  • Major (incompatible API changes)

And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.

  • No thanks, I would rather do it manually 😞

@vtexdaniel vtexdaniel requested review from carolinamenezes, icazevedo, igorbrasileiro and mullerjk and removed request for a team June 26, 2021 15:21
@vtex-io-docs-bot

Copy link
Copy Markdown

Beep boop 🤖

Thank you so much for keeping our documentation up-to-date ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant