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

[Bug]: Unable to Use On Intel Mac #2773

Closed
mtdvlpr opened this issue Oct 18, 2024 · 50 comments
Closed

[Bug]: Unable to Use On Intel Mac #2773

mtdvlpr opened this issue Oct 18, 2024 · 50 comments
Labels
bug Something isn't working implemented Taken care of by a new commit/version macOS Specific to macOS

Comments

@mtdvlpr
Copy link
Collaborator

mtdvlpr commented Oct 18, 2024

Hi Brothers,

I really like the look and feel of the new interface. I do have some observations, and maybe a bug. Including screenshots to help:

  1. On launch, there is an immediate error that nothing could be retrieved for the meeting dates:
    1.png
These issues have been fixed already 3. When I imported the files I had gotten from another Mac, the results don’t stick to the numbering system which used to put them in the correct order. (See screenshots of how it looks before download and after download) ![2.png](https://github.com/user-attachments/assets/e76d2677-12e6-4655-93eb-b26747573bc9)
  1. This is how they download from DropBox folder (Notice not in correct order, even ignoring the numbering system):
    3.png

After manual change to ordering of files:
4.png

  1. Notice too that there isn’t a heading for the WT study on this view. But that might be because it didn’t automatically download anything.

  2. Also, when I try to move the songs into the correct order, they can’t be dragged to the top of the screen, and then the screen automatically scroll to the place I want to inset it. It takes several moves to get the media to where it needs to land.

  3. Is there a list of keyboard shortcuts that would allow moving to the next piece of media rather than manually selecting it?

  4. Also, noticed that you can only have the small window or full screen rather than being able to set the screen to the size the user might need. Probably necessary when doing a hybrid ZOOM meeting.

I wasn’t able to make this version download any media for any future meeting, and that includes this Sunday and next Tuesday. Not seeing any interface that would allow manual selection of a date for media. If you use the calendar, it will return this:
5.png

We won’t have any need for VLC with this new version. Just need a little tweaking possibly? I appreciate all your great work on this wonderful tool!

Thanks,
Mark Cillis
@.***

On Oct 17, 2024, at 10:02 AM, Manoah Tervoort @.***> wrote:

Note that the congregation sync feature is not present in the upcoming big release (v24.10.x) and up.


Reply to this email directly, view it on GitHub #2709 (reply in thread), or unsubscribe https://github.com/notifications/unsubscribe-auth/A6S45XGLTO2VJ6QZMMFB7OLZ37NQNAVCNFSM6AAAAABP2NVXXCVHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTAOJXGM3DMNY.
You are receiving this because you were mentioned.

Originally posted by @skimark1 in #2709 (comment)

@mtdvlpr mtdvlpr added the bug Something isn't working label Oct 18, 2024
@mtdvlpr mtdvlpr changed the title Observations and maybe a bug [Bug]: Observations and maybe a bug Oct 18, 2024
@alanfermtz
Copy link

alanfermtz commented Oct 18, 2024

The first screenshot happens to me if I finished my setup, then go clear cache, and return to multimedia page

@skimark1

This comment was marked as outdated.

@sircharlo

This comment was marked as outdated.

@skimark1

This comment was marked as outdated.

@mtdvlpr
Copy link
Collaborator Author

mtdvlpr commented Oct 20, 2024

@skimark1, you can still download the previous version and in the settings disable automatic updates. Once the problem has been solved, we'll notify you and you can safely upgrade again.

@sircharlo

This comment was marked as outdated.

@skimark1

This comment was marked as outdated.

@sircharlo

This comment was marked as outdated.

@mtdvlpr mtdvlpr added the in-progress Currently working on this label Oct 21, 2024
@sircharlo

This comment was marked as outdated.

@skimark1

This comment was marked as outdated.

@sircharlo

This comment was marked as outdated.

@skimark1

This comment was marked as outdated.

@sircharlo

This comment was marked as outdated.

@skimark1

This comment was marked as outdated.

@sircharlo
Copy link
Owner

Is this an Intel Mac?

@skimark1
Copy link

skimark1 commented Oct 22, 2024 via email

@skimark1
Copy link

skimark1 commented Oct 22, 2024

I've got a 2023 MacBook Air M2 series, and M3 works fine on it. Must be an Intel thing?

@sircharlo

This comment was marked as outdated.

@sircharlo sircharlo changed the title [Bug]: Observations and maybe a bug [Bug]: Unable to Use On Intel Mac Oct 22, 2024
@skimark1

This comment was marked as outdated.

@sircharlo
Copy link
Owner

Ok, thank you. I'll see how to force a recompilation for Intel Macs of the module that's breaking.

@skimark1
Copy link

skimark1 commented Oct 22, 2024 via email

@sircharlo
Copy link
Owner

If only it can work on Intel Macs now :)

@Jiang-TJ
Copy link

Jiang-TJ commented Nov 4, 2024

Hope a solution to this intel mac problem can be found. Thanks friends for your hard work, it really means a lot!

@mtdvlpr
Copy link
Collaborator Author

mtdvlpr commented Nov 4, 2024

I found a post that might be related:

WiseLibs/better-sqlite3#126 (comment)

@mtdvlpr mtdvlpr added the help wanted Extra attention is needed label Nov 4, 2024
@mtdvlpr
Copy link
Collaborator Author

mtdvlpr commented Nov 4, 2024

Hi @skimark1, could you try the below version for me?

https://we.tl/t-sAEBOeDUiw

@mtdvlpr
Copy link
Collaborator Author

mtdvlpr commented Nov 4, 2024

@skimark1, I also have another one that might give less security issues:

https://we.tl/t-hV16x8Wa8r

@skimark1
Copy link

skimark1 commented Nov 4, 2024 via email

@sircharlo
Copy link
Owner

That works!!! Thanks, Mark Cillis

Just to be clear, you mean that the app fully works as expected on an Intel Mac?

@mtdvlpr
Copy link
Collaborator Author

mtdvlpr commented Nov 4, 2024

@skimark1, I also have another one that might give less security issues:

https://we.tl/t-hV16x8Wa8r

This one is built from https://github.com/sircharlo/meeting-media-manager/tree/fix/macos-intel

@mtdvlpr
Copy link
Collaborator Author

mtdvlpr commented Nov 4, 2024

@skimark1, does everything work as expected now? Or did it only solve the security warning?

@skimark1
Copy link

skimark1 commented Nov 4, 2024 via email

@sircharlo
Copy link
Owner

That one still has the security warning. Thanks, Mark Cillis

But does the app run as it should?

@skimark1
Copy link

skimark1 commented Nov 4, 2024 via email

@mtdvlpr
Copy link
Collaborator Author

mtdvlpr commented Nov 4, 2024

@skimark1, thank you for helping us pinpoint the problem and resolve it!

@sircharlo
Copy link
Owner

Everything works as expected, just get the security challenge to be able to open it the first time.

This is great news!!

@sircharlo
Copy link
Owner

Thanks for your hard work @mtdvlpr !!

@mtdvlpr mtdvlpr added awaiting-release Will be released with next update and removed in-progress Currently working on this help wanted Extra attention is needed labels Nov 4, 2024
@sircharlo sircharlo added implemented Taken care of by a new commit/version and removed awaiting-release Will be released with next update labels Nov 6, 2024
@mtdvlpr
Copy link
Collaborator Author

mtdvlpr commented Nov 6, 2024

Important

You might have to install the new version manually instead of automatically, due to a bug that's been fixed for upcoming releases.

@skimark1
Copy link

skimark1 commented Nov 6, 2024 via email

@mtdvlpr
Copy link
Collaborator Author

mtdvlpr commented Nov 6, 2024

@skimark1, correct me if I'm wrong, but the default behaviour for macos is to keep the app running even if all windows are closed. You can activate thr app again by simply clicking the M3 icon once more. This should create another window again.

@sircharlo
Copy link
Owner

One thing I noticed is that if you click the red ball button, it doesn’t quit M3. Instead it is left running and I don’t see a way to bring the window back without actually quitting the app and relaunching. Might be confusing for some? Love the progress made! Thanks, Mark Cillis @.***

@mtdvlpr This is why i had removed the macOS check here:

if (PLATFORM !== 'darwin') app.quit();

Do you think we should either revert to quit when windows are closed, or to recreate the main window on activate?

@skimark1
Copy link

skimark1 commented Nov 6, 2024 via email

@mtdvlpr
Copy link
Collaborator Author

mtdvlpr commented Nov 6, 2024

@skimark1, I gound the bug and fixed it.

@sircharlo, I think it's better to stick with the system default behaviour. That's (I assume) what macos users are accustomed to. But if our macos users prefer actually closing m3 when the windows are closed, then let's do that.

@sircharlo
Copy link
Owner

If you found the bug I'm happy to leave the behavior as is.

@skimark1
Copy link

skimark1 commented Nov 6, 2024 via email

@mtdvlpr
Copy link
Collaborator Author

mtdvlpr commented Nov 6, 2024

@skimark1, yes. The bug has been fixed and will be in the next release.

@skimark1
Copy link

skimark1 commented Nov 6, 2024 via email

@mtdvlpr
Copy link
Collaborator Author

mtdvlpr commented Nov 7, 2024

@skimark1, the yeartext screen can have a custom background image, but it is not persistent anymore, so every time you restart M3, the background is reset to the yeartext. This is intended behavior, because the guidelines specify the yeartext should be shown during regular meetings (unless it's a special occasion like the memorial). Days like the memorial is why the custom background feature exists.

@sircharlo
Copy link
Owner

@skimark1 To add to @mtdvlpr's excellent answer, we simply try to replicate known behaviors as much as possible and not deviate from what one would normally expect to see. There's much to be said for a consistent look, no matter which meeting one attends, anywhere in the world!

@mtdvlpr mtdvlpr added the macOS Specific to macOS label Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working implemented Taken care of by a new commit/version macOS Specific to macOS
Projects
None yet
Development

No branches or pull requests

5 participants