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

Releasing 2.0.0 #267

Closed
16 tasks done
whyboris opened this issue Aug 26, 2019 · 20 comments
Closed
16 tasks done

Releasing 2.0.0 #267

whyboris opened this issue Aug 26, 2019 · 20 comments
Assignees
Labels
2.0.0 Priority for v2.0.0 release
Milestone

Comments

@whyboris
Copy link
Owner

whyboris commented Aug 26, 2019

We are in the last week or so before the official 2.0.0 release 🎉

Closing #234 and moving the leftover issues here for cleanliness 👍

Fixed bugs:

Last steps:

@whyboris whyboris added the 2.0.0 Priority for v2.0.0 release label Aug 26, 2019
@whyboris whyboris added this to the v2.0.0 milestone Aug 26, 2019
@whyboris whyboris self-assigned this Aug 26, 2019
@whyboris whyboris pinned this issue Aug 26, 2019
@whyboris whyboris mentioned this issue Aug 26, 2019
16 tasks
@whyboris
Copy link
Owner Author

whyboris commented Aug 27, 2019

#268 - Everything working smoothly

  • app installer is ~70MB for Windows
  • runs with no errors after installation ✅

@whyboris
Copy link
Owner Author

#269 - super huge refactor of extract-screenshots functionality 😅

  • should give enough time to extract each thumb/strip/clip/clip thumb
  • should not get hung up on corrupt files (there's now a timeout which kills ffmpeg if it takes too long)

@whyboris
Copy link
Owner Author

whyboris commented Aug 30, 2019

Huge thank you to people working on Electron for swiftly releasing a patch, unblocking this release 🙇
electron/electron#19816 fixed with electron/electron#20001 ❤️

@JayVem
Copy link

JayVem commented Sep 29, 2019

hi boris, is the new release happening soon?

@whyboris
Copy link
Owner Author

Thank you for checking in -- I'm busy finishing up a charity-related project this and next week; I hope to release Video Hub App by November 1st now 😅

Thanks to everyone for your patience 🙇

@JayVem
Copy link

JayVem commented Sep 29, 2019

thank you, appreciate your quick response. Looking forward to the new release.

@whyboris
Copy link
Owner Author

😓 finally finished the damn icons: #283 😠 -- making icons is no fun for me

@whyboris
Copy link
Owner Author

whyboris commented Oct 11, 2019

⚠️ At this point I'm soliciting bug reports before the final release -- I'm hoping to get a final build this weekend 🙇 would appreciate any kind of comments too 🙇

I'm aware of 2 major bugs:

@whyboris
Copy link
Owner Author

Moved some minor (non critical) tasks to #284 - for after release. Let's get this release out 🚀 😆

@whyboris
Copy link
Owner Author

I think I fixed the path issues, closing another bug 🎉 with #286

@whyboris
Copy link
Owner Author

🚀 🚀 🚀 🚀 🚀 🚀 🚀 🚀 🚀 🚀 🚀 🚀
Let's release 🤞

@whyboris
Copy link
Owner Author

🛑 Nope 😅
Turns out this bug is not fixed: #270 (comment)

When you import the root of an external hard drive and try to navigate using the folder navigation, folders show up as empty when clicking inside them 😢

Will fix 👌 ... release delayed until next weekend 🐌

@LordJon
Copy link

LordJon commented Oct 14, 2019

I hope I'm not cluttering the overview here but I just wanted to say this. I've been lurking around and following changes here and on the fixes made for Electron and such. It all sounds very organised. Compliments to all of you.

Also good luck with the last fixes and the backlog for after release. Looking forward for 2.0! 😃

@whyboris
Copy link
Owner Author

🏷 #287 may have fixed the last major bug with the current release. I'm pretty sure I can release as is (though don't have time until the weekend)

@cal2195 if you are able to, by mid-weekend let me know what happened with #269 (comment) -- maybe I can fix that up as well before release 🙇

No worries if you're unable 👌

@whyboris
Copy link
Owner Author

Cal responded: #269 (comment)

Seems like on the videos above 1080p the timeout is not long enough and the filmstrip extraction process doesn't finish.

So setExtractionDurations method that where I roughly guessed about timeout durations should be changed. Specifically, the filmstrip computation:

https://github.com/whyboris/Video-Hub-App/blob/master/main-extract.ts#L451

I used a computation taking thumbnail height as a factor in a linear rather than square fashion - a simplification that seems to backfire at higher resolutions. Should be a straightforward fix 👌

@whyboris
Copy link
Owner Author

whyboris commented Oct 15, 2019

Before 2.0.0 release it may be worth replacing the complicated pipe that inserts the icon with the character 🎉

This will allow me to remove the leftover font from a while back, and remove the pipe -- will speed things up in the app (an imperceptible amount 😅 )

edit: nope -- tried it, looks ugly 💩 -- keeping the pipe and the icon 👌

@whyboris
Copy link
Owner Author

I updated the method that provides timeouts for killing ffmpeg when extracting takes too long:
0cd073d
⚠️ I'm not happy about how I'm handling it, but it's likely better than before 🤷‍♂

@whyboris
Copy link
Owner Author

Released 😂

@whyboris whyboris unpinned this issue Oct 18, 2019
@LordJon
Copy link

LordJon commented Oct 18, 2019

Hi Boris, that was fast. Going to test the new 2.0 hub later today. I can't wait. Just a small note, your blog says that on October the 12th VHA 2.0 was released, probably the previous release date you had in mind before the external drive problem appeared haha.
Cheers!

@whyboris
Copy link
Owner Author

Thank you @LordJon -- I did forget to update the date on the Blog 😅

I don't feel like I've 100% polished the app, but at this time I think it's safe to release and will take only a little patching up to make it as excellent as I'd like it to be.

Really looking forward to all the feedback 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.0.0 Priority for v2.0.0 release
Projects
None yet
Development

No branches or pull requests

3 participants