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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Video Pages #369

Merged
merged 6 commits into from Sep 24, 2019

Conversation

@dirkkelly
Copy link
Member

commented Sep 9, 2019

Completed the "big refactor" of the website, lots of improvements in this one.

Everyone your review 馃憖 and feedback 馃摙 for this change would be really appreciated, please leave comments on the pull request #369

Video https://youtu.be/mYkkupSItVM

For Users

For Creators

For Developers

  • Removes data files, now everything is edited in content
  • Includes migration scripts ./scripts/migrate.sh for moving bake cli content
  • Uses archetypes and YQ scripting to generate video pages.
  • Removes need for videos array on channel pages (not removed yet)
  • Slow build time is a result of all the content, there's probably room for optimization
  • Search is slow and janky, could do with a refactor

This puts the website in a good position for go live, and allows us to focus more on

  • Community driven content updates (onboarding, and support by @dirkkelly and volunteers to help with updating)
  • Deployment of a PeerTube instance (in research with @samirillion#9403)
  • Redesign of the website (in design with @litgarbo#6683)
  • Improvement of the javascript framework (icebox with @samirillion#9403 )
  • Discussion about content management tools (in research with @dirkkelly#135)

Checks


Build: 2m7.6s
Deploy: 6m40s

@dirkkelly dirkkelly self-assigned this Sep 9, 2019
@dirkkelly dirkkelly force-pushed the videos branch from b828b43 to fece682 Sep 9, 2019
@dirkkelly dirkkelly force-pushed the videos branch from b67020a to 67ae27b Sep 15, 2019
dirkkelly added 2 commits Sep 15, 2019
@dirkkelly dirkkelly added this to In progress in Launch via automation Sep 15, 2019
@dirkkelly dirkkelly requested review from breadtubetv/development Sep 15, 2019
@j16r
j16r approved these changes Sep 23, 2019
Launch automation moved this from In progress to Reviewer approved Sep 23, 2019
@dirkkelly dirkkelly merged commit 0a47c86 into master Sep 24, 2019
3 of 5 checks passed
3 of 5 checks passed
Header rules - breadtubetv No header rules processed
Details
Pages changed - breadtubetv 2530 new files uploaded
Details
Mixed content - breadtubetv No mixed content detected
Details
Redirect rules - breadtubetv 12 redirect rules processed
Details
netlify/breadtubetv/deploy-preview Deploy preview ready!
Details
Launch automation moved this from Reviewer approved to Done Sep 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Launch
  
Done
2 participants
You can鈥檛 perform that action at this time.