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

faq: add tail emission/nodes video #2244

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

plowsof
Copy link
Contributor

@plowsof plowsof commented Feb 8, 2024

No description provided.

Copy link

netlify bot commented Feb 8, 2024

Deploy Preview for barolo-time-757cf9 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 19e203d
🔍 Latest deploy log https://app.netlify.com/sites/barolo-time-757cf9/deploys/6616ecdac87a330008b669d0
😎 Deploy Preview https://deploy-preview-2244--barolo-time-757cf9.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@nahuhh nahuhh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Monero_Tail_Emission-VOSTO_EMISIO.mp4

please make a habit of using - and _ and not [space] for filenames. 🙏😭

nobody likes %20 in their filename

@plowsof plowsof force-pushed the video-tail-emission branch 4 times, most recently from 52e98e5 to c090cbc Compare February 8, 2024 18:31
@plowsof
Copy link
Contributor Author

plowsof commented Feb 8, 2024

im not willingly putting spaces / %20 in filenames (and having to forcepush about 5 times). when a pull request is made, you generally try to blend in / use similar code styles/syntax as the area you are changing if there are no strict standards in place

@nahuhh
Copy link
Contributor

nahuhh commented Feb 8, 2024

Yes yes, but the filename is bad (spelling and an extra space at the end), even if it use %20 like other stuff in the list

@nahuhh
Copy link
Contributor

nahuhh commented Feb 8, 2024

Yes yes, but the filename is bad (spelling and an extra space at the end), even if it use %20 like other stuff in the list
edit: and its an mp4, the rest are m4v

(i have np with mp4, as long as it works reliably on, say, ios devices). Not sure why m4v was chosen before, perhaps just personal preference

@plowsof
Copy link
Contributor Author

plowsof commented Feb 8, 2024

spelling/extra space corrected thanks, i requested an m4a and got an mp4, lets wait for vosto emisio to send an m4a (also no idea about these formats)

@plowsof
Copy link
Contributor Author

plowsof commented Feb 10, 2024

took a look at what M4V is and.. its not good:

M4V is a container video format developed by Apple, closely resembling the MP4 format. The main difference between M4V and MP4 is that an M4V file can enjoy DRM copy protection. This DRM protection is in place to prevent piracy, ensuring that M4V files purchased from the iTunes store cannot play on non-Apple players.

we will definitely not be getting an M4V version of the tail emission video from vosto :D

@plowsof plowsof force-pushed the video-tail-emission branch 2 times, most recently from 682e330 to 9e6a04d Compare February 10, 2024 19:04
@plowsof
Copy link
Contributor Author

plowsof commented Feb 11, 2024

after realising what m4v format is, i dont think we should blend in anymore and stop using m4v. i have also replaced spaces with "_" in the filename. The only concern now could be that the filesize is 40~MB

@HardenedSteel
Copy link
Contributor

WebM can be even better.

@VOSTO-EMISIO
Copy link

Hey Everyone,

Here are .webm files for both the Tail Emission video and the Nodes video. We've managed to compress the videos by about ~36% per video without losing quality. If someone can compress even further, please go ahead and do so.

Here are the files: https://drive.proton.me/urls/04APQES9A0#Ar5KSqfDdE3Y

@plowsof
Copy link
Contributor Author

plowsof commented Apr 10, 2024

Thanks Vosto. I've added the 2 webm files. instead of adding a new translated string for nodes, ive recycled one for "Node" hopefully this is OK

@plowsof plowsof changed the title faq: add tail emission video faq: add tail emission/nodes video Apr 10, 2024
Copy link
Contributor

@rottenwheel rottenwheel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me; website can use some more up-to-date videos.

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

Successfully merging this pull request may close these issues.

None yet

5 participants