Skip to content
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

Try breaking out of Vue to fix Safari issue #12

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jamsea
Copy link
Contributor

@jamsea jamsea commented Oct 7, 2022

I attempted to break the daily call experience out into as vanilla as a javascript as possible, but I'm still getting the same error when I disconnect my airpods, reconnect, and disconnect again (screen shot below).

I thought this might have something to do with how mounted() works in vue, however I've confirmed that it's only called once.

I'm really scratching my head on this one 🤔

image

@jamsea jamsea changed the title Hush/safar update vue Try breaking out of Vue to fix Safari issue Oct 7, 2022
@jamsea jamsea force-pushed the hush/safar-update-vue branch 2 times, most recently from c54d57b to 70c83cf Compare October 17, 2022 07:45
@jamsea jamsea changed the base branch from hush/update-vue to main October 17, 2022 07:45
* Update to vite, use latest vue tools

Remove comment

Update babel and package-lock

Fix lint issues

Fix lint issues

Fix prettier

Add github action

Test before building

* Lint ts files too

* Add tsconfig

Lint ts files too

Add tsconfig

Add tsconfig

Use vanilla js in Vue does not work

Still not working

Add console log for vue mounted

Gate off and blocked instead of playable

THIS WORKS.

Move div back into Vue. THIS WORKS

Fix spacing
@jamsea jamsea force-pushed the hush/safar-update-vue branch from 70c83cf to 8952e39 Compare October 17, 2022 07:51
@netlify
Copy link

netlify bot commented Oct 17, 2022

Deploy Preview for vue-call-object ready!

Name Link
🔨 Latest commit 8871dca
🔍 Latest deploy log https://app.netlify.com/sites/vue-call-object/deploys/634d873e6336fc0009885fdb
😎 Deploy Preview https://deploy-preview-12--vue-call-object.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

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