-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Support for saving recorded video to Nextcloud #192
Comments
This isn't about this issue, but I need to say it: You did a really good job on this! It looks briliant and seems to work as a charm. Thanks for you invested time and energy! I tried a lot of commercial alternatives, but I think this is an ideal candidate for most users. So, it's very impressive, that it's created by just one person and offered for free with MIT license. There only a few things I would need to add - ability to move circle with camera output and a way to upload final video to my Nextcloud. I think I could help with this feature, but it seems like you have just an older version of this project shared publicly. Btw, do you accept pull requests with translations to new languages in general? I could add Czech one, if you confirm it. |
I'm considering adding custom targets for saving recorded videos in the new version, but it's not something I've worked on just yet. Regarding the code for MV3, yes, it is local. The matter is a bit more complex as I am adding a paid cloud component to the extension (among other features, for easier collaboration, sharing, etc.) and I need to keep the free self-hostable version contained. But I'm hoping by the time it is released (which shouldn't be long now, I've completed the end to end recording flow, I'm on testing + bug hunting stage right now!) I'll have the updated codebase ready on GitHub. As per translations, I'm not accepting them right now since the UI is completely different, so unfortunately it would have to be redone. I'm still looking into how best to approach that, given the large number of users using Screenity in different languages, but I will at least have some languages covered on release. |
Thank you for your answer! I completely understand. So, I'll wait for the release of the new version. If you want to have a Czech translation in the new release, you can contact me (or tag me) to help. I care a lot about the quality of my translations, so I would need at least a wireframes (or Figma, etc.) with translation codes or (preferably) a prototype of the app. However, I try to support my translations, so you'll receive updates as soon as possible (usually a few days) after releasing a new version of the English version. Smaller projects generally use JSON/YAML files managed by pull requests (e.g. Saber or Scanservjs). Bigger ones use Webplate/Transifex/Crowdin or similar software (e.g. Nextcloud or FairEmail) with regular syncing to the project repository. Both ways are usable (even for the long term), but the second one allows for contributions from less technically oriented people, so there are possibly more contributors. It is a more complicated to deal with translations stored elsewhere (you need to set up some synchronization mechanism, etc.). Anyway, I wish you a good luck with development! |
I've added the MV3 code now here in GitHub. I will look into better ways to handle translations as the codebase is big, and there's a fair bit of text. Will look into Nextcloud soon. |
Thanks for your work! I really appreciate it! Good luck with the next development of this great project! I just read info about the new version and I'm gonna try it. After you handle the translation system, feel free to tag me and I'll try to make a Czech translation for you, so other translators will be motivated to translate it too 😉. |
Do you plan to support Nextcloud (or maybe general WebDAV storage) as other possible target for saving recorded videos?
Btw, you noted something about changes you have already prepared for the next update, but I didn't find anything in this GitHub repository. So, do you have it just locally? Could you share that before official release?
The text was updated successfully, but these errors were encountered: