InfiniTime icon indicating copy to clipboard operation
InfiniTime copied to clipboard

Published 20 hours ago • verified publisher icon InfiniTimeOrg

Adding Documentation for tips on adding new watch face

Open bmsleight opened this issue 3 years ago • 5 comments

closes #1301

bmsleight avatar Aug 28 '22 11:08 bmsleight

Thanks for the feedback - updating now.

bmsleight avatar Aug 28 '22 12:08 bmsleight

Since this contains no details of the new watch face, I think it would be better to give the files a more generic name like NewWatchFace.cpp.

Edit: 😊👍 for writing docs!

minacode avatar Aug 29 '22 17:08 minacode

Hi @minacode - Thanks for the feedback.

I think it would be better to give the files a more generic name like #1306 PR has details of the new watch face.

But happy to change it to NewWatchFace.cpp

Which is preferred 1) Change to NewWatchFace.cpp or 2) Explicit reference to the details of the new watch face ? (I think I prefer 1 - but i have tested the docs in 2 and are mindful of adding an untested error)

bmsleight avatar Aug 29 '22 17:08 bmsleight

I vote for 1.

minacode avatar Aug 29 '22 17:08 minacode

Grateful for some guidance on what is needed or missing ?

bmsleight avatar Sep 12 '22 18:09 bmsleight