Features and Fixes Part 2

[nlogozzo]

Closes #363
Closes #365
Closes #375

Blocking:

• Unable to set PublishingDate for flac files Zeugma440/atldotnet#235

[nlogozzo]

@kissthermite
You can download and install the .flatpak file from the assets section on this CI build: https://github.com/NickvisionApps/Tagger/actions/runs/6601503583 to test the remembering of format strings and the sorting files by album with disc number while we wait for our backend's (atldotnet) developer to fix the flac file issue.

Just a note, the format string used for file name to tag will be remembered independently of that used by tag to file name.

Once you download and extract the .flatpak file from the assets, you can use these commands to install it:

#from the directory containing the .flatpak file
flatpak install org.nickvision.tagger-x64.flatpak
flatpak run org.nickvision.tagger//master

Once you've tested and are done with it (or you can keep using it until the next beta release) you can uninstall it with:

flatpak uninstall org.nickvision.tagger//master

Feel free to comment with any issues, comments, questions, or concerns :))

[kissthermite]

Feel free to comment with any issues, comments, questions, or concerns :))

It does remember my custom format string even after closing Tagger. Such a nice QOL. Good job @nlogozzo!

[kissthermite]

Feel free to comment with any issues, comments, questions, or concerns :))

And album sorting now behave the way i expected too. Another job well done!

[nlogozzo]

@kissthermite Glad everything works as expected :)

This build should contain the fix for PublishingDate and flac files: https://github.com/NickvisionApps/Tagger/actions/runs/6603980868

Once you confirm that works I will merge this PR and release RC1 and we should be all good for a stable release on Friday!

[kissthermite]

@kissthermite Glad everything works as expected :)

This build should contain the fix for PublishingDate and flac files: https://github.com/NickvisionApps/Tagger/actions/runs/6603980868

Once you confirm that works I will merge this PR and release RC1 and we should be all good for a stable release on Friday!

it worked! 🚀

I just realized that there's currently no way to remove Publishing Date tag?

[nlogozzo]

Yay!! I have to go to work now but once I get home later td I'll merge and do a release :)

[kissthermite]

Yay!! I have to go to work now but once I get home later td I'll merge and do a release :)

I just edited my comment in case you didn't see it.

I just realized that there's currently no way to remove Publishing Date tag?

That trash can button can be seen on Custom Properties
VORBIS-VENDOR would do nicely beside Pick a date/Date button

[nlogozzo]

That trash can button can be seen on Custom Properties
VORBIS-VENDOR would do nicely beside Pick a date/Date button

Ah, good idea! Will implement that when I get home instead

[nlogozzo]

That trash can button can be seen on Custom Properties
VORBIS-VENDOR would do nicely beside Pick a date/Date button

Ah, good idea! Will implement that when I get home instead

@kissthermite Ok, done! Use this build to test: https://github.com/NickvisionApps/Tagger/actions/runs/6605232433

[fsobolev]

I think we should take disc number to consideration everywhere where we compare track numbers: for sorting by artist, year and genre too.

[nlogozzo]

I think we should take disc number to consideration everywhere where we compare track numbers: for sorting by artist, year and genre too.

Done in latest commit!

[kissthermite]

---

@nlogozzo another job completed fantastically!

[nlogozzo]

@kissthermite RC1 will come in a little bit.

Just be advised that configurations are kept independent of branches. Meaning flatpak beta and flatpak master will be different. So upon installing rc1, your format string won't be remembered until you use it for the first time on the version.

[fsobolev]

Just be advised that configurations are kept independent of branches. Meaning flatpak beta and flatpak master will be different. So upon installing rc1, your format string won't be remembered until you use it for the first time on the version.

Why is that? Flatpak data dir is always ~/.var/app/org.nickvision.tagger no matter the branch

[nlogozzo]

Just be advised that configurations are kept independent of branches. Meaning flatpak beta and flatpak master will be different. So upon installing rc1, your format string won't be remembered until you use it for the first time on the version.

Why is that? Flatpak data dir is always ~/.var/app/org.nickvision.tagger no matter the branch

I could be wrong. I thought the whole branch was kept separate from each other. Guess it's just the binaries and the rest is shared.