Checklist before releasing 22.11.0

[mtdvlpr]

Some things that need to be tested again before releasing:

• Malagasy works without dayjs error
• Image zoom and pan works correctly and preview and media window always show the same (3399c9a)
• Everything works correctly on Windows 32-bit platform (b92db40)
• mp3 to mp4 conversion works correctly (412238b)

[sircharlo]

I'll test the following on Win32:

• sync
• cong sync
• cong upload
• cong delete
• image convert of PDFs
• MP4 convert of images and MP3s (MP3s not working)
• media playback window
• zoom/pan
• background music

[mtdvlpr]

I'll test the following on Win32:

Awesome! Afterwards I think we're good to go with the release :)
The Sentry issues that are still open can't be pinned down to a fix (yet), so I'll set them to resolved in next release.

[sircharlo]

Slight regression introduced, see #634 (comment)

[mtdvlpr]

Slight regression introduced, see #634 (comment)

Fixed in #635

[mtdvlpr]

I got my virtual Windows 32-bit machine working too! Everything seems to work, except converting mp3 to mp4. That goes wrong every time. The error is now caught and a human friendly warning is given. Since converting mp3 to mp4 is a real edge case, and it only goes wrong on (low-memory?) 32-bit machines, I'd say forget about it.

[sircharlo]

I got my virtual Windows 32-bit machine working too! Everything seems to work, except converting mp3 to mp4. That goes wrong every time. The error is now caught and a human friendly warning is given. Since converting mp3 to mp4 is a real edge case, and it only goes wrong on (low-memory?) 32-bit machines, I'd say forget about it.

Even worse, it works for me in Win32 😆
But agreed, not the end of the world.

[mtdvlpr]

Haha, then it's probably a memory thing.

[mtdvlpr]

I'd say we're good to go then

[sircharlo]

Will you do the honors, or me?

[mtdvlpr]

It's easier for you, since you can push directly to master. I'd have to create a pull request which I'm not sure has the desired outcome.

[mtdvlpr]

@sircharlo, so what happened is you wanted to push the release commit, but got blocked because you had to git pull first. Then when you pushed the merge commit got evaluated instead of the release commit, so no release workflow was executed. You'll have to do another chore(release)

[sircharlo]

@sircharlo, so what happened is you wanted to push the release commit, but got blocked because you had to git pull first. Then when you pushed the merge commit got evaluated instead of the release commit, so no release workflow was executed. You'll have to do another chore(release)

Yeah my bad, I redid it and it worked after. :)

[sircharlo]

Weirdly enough, the autoupdate didn't work for me until I deleted the %localappdata%\meeting-media-manager-updater folder.

[mtdvlpr]

Hmm, it worked fine for me. The desktop shortcut also correctly updated this time.

[mtdvlpr]

Did you have a locally generated version installed by chance?

[sircharlo]

Nah, I didn't build on my machine at all. Must be a glitch with my setup, I wouldn't worry about it too much