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

Better use of wifi and time configuration #9

Closed
wants to merge 8 commits into from

Conversation

baronunread
Copy link

I've been hard at work but this is a possible way to make this entire system last a bit longer than what it does currently. There's some missing features for sure but I'll work on it some more when I do have the time.

With this fork I've added:

Not everything will fit with your vision and that's fine, send me some comments if you'd like

@qewer33 now for a personal message to you: I've been working on it because I fully believe that there's so much more to this and the design and idea are just fantastic. We can definitely squeeze a lot more from this so I hope at least in some way I reignited your passion about it :)

- currently disabled vibrating motors for future functionality
…g up.

- added handling of stored WiFi credentials and os_config backups. If none are found use last saved time for RTC
- added new charging state to avoid continuous loss of connection while charging
- add more comments
@baronunread
Copy link
Author

I don't want to modify the os_config.h all the time before committing I'm making an empty one and an example

@qewer33
Copy link
Owner

qewer33 commented Sep 13, 2024

Hey! Thank you for the PR, and about the message, I've kinda moved on to other stuff and projects these days so I'm not sure if I will be able to continue working on qpaperOS that much. You are definitely better off just maintaining your own fork (and if you're committed to actively develop it for some time, I can put a redirect to your fork on top of the README :D).

Also as a review for this PR, you should probably not untrack os_config.h since the firmware will not compile without it + it's good to provide a default config.

@baronunread
Copy link
Author

Ah yeah I figured I messed up when I untracked lol
I added it back in and did this command git update-index --assume-unchanged src/os_config.h

Thank you for your time and for the offer, if you think it's a worthy successor to your work then I'be quite happy and will continue to nibble on it little by little. Are you still using the watch? I was thinking of adding the missing features (minus the GPS, that would really be a battery hog so I didn't get that model haha!)

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.

2 participants