code4recovery / online-meeting-list

Timezone-agnostic meeting finder for online-only meetings
https://online-meeting-list.netlify.app
MIT License
8 stars 8 forks source link

Phone String #72

Closed twn2twn closed 1 month ago

twn2twn commented 1 month ago

Hi, Jeff OIAA -

I was under the impression the 'conference URL' had the phone string format -

PhoneNumber,,ZoomID#,,Passcode#

My laptop can't dial out, but when I use the PHONE button from the site on my phone, it displays all number together without the symbols and doesn't work.

This is how most all 'one touch' dial ins were set up and I followed suit. Is this not correct? If correct, how can it be made to work? thank you

joshreisner commented 1 month ago

@twn2twn thanks for spotting that. looks like the meeting finder has always been stripping out all non numeric characters, even though as you say the preferred format for most sites is to make a one-tap URL like this: +12125551212,,123456789#,,#,,444444# - i added this instruction to the sidebar on central relatively recently which is probably why most of the meetings don't have them yet - and probably why no one noticed

but here's an example that does: https://aa-intergroup.org/meetings/the-acid-test-6

should be a pretty easy code change on the frontend, i'll transfer this issue there

joshreisner commented 1 month ago

@mattbylett if you could pull and redeploy this issue should be fixed now ✅

mattbylett commented 1 month ago

Thanks Josh - will do 👌

We help businesses develop integrated ecommerce digital solutions

Matthew Bylett Founder Think New Ltd 022 029 2881

On Sun, 13 Oct 2024 at 7:24 AM, Josh Reisner @.***> wrote:

@mattbylett https://github.com/mattbylett if you could pull and redeploy this issue should be fixed now ✅

— Reply to this email directly, view it on GitHub https://github.com/code4recovery/online-meeting-list/issues/72#issuecomment-2408651579, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHNVISISBI7EOVFRONXQ5HDZ3FSMJAVCNFSM6AAAAABP2UFIEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBYGY2TCNJXHE . You are receiving this because you were mentioned.Message ID: @.***>

mattbylett commented 1 month ago

Pulled and deployed - ready for testing

joshreisner commented 1 month ago

thanks @mattbylett !

@twn2twn let us know if that fixes it or if not what meeting to test it on, thanks

twn2twn commented 1 month ago

Works for me! thank you both, Jeff B

On Sat, Oct 12, 2024 at 2:36 PM mattbylett @.***> wrote:

Pulled and deployed - ready for testing

— Reply to this email directly, view it on GitHub https://github.com/code4recovery/online-meeting-list/issues/72#issuecomment-2408654942, or unsubscribe https://github.com/notifications/unsubscribe-auth/BGUR4JZ6ABHE7NKGCX7GUEDZ3FT2XAVCNFSM6AAAAABP2UFIEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBYGY2TIOJUGI . You are receiving this because you authored the thread.Message ID: @.***>