infidel-sensor icon indicating copy to clipboard operation
infidel-sensor copied to clipboard

Added 'firmware' section to docs

Open leonhma opened this issue 4 years ago • 7 comments
trafficstars

This is again some changes to the docs. Can be seen on my forked repo leonhma/infidel-sensor. I also deleted the gh-pages workflow, in favor of hosting the docs on rtd. This is also linked in the forked repo and the pullrequest preview as in #44 can be seen in leonhma/infidel-sensor#2 (click on 'show all checks' and then 'details'). To add this feature either an existing maintainer has to link to rtd and set stuff up, or i can do it and will have to be made maintainer.

leonhma avatar Jul 11 '21 10:07 leonhma

@leonhma I don't mind making you a maintainer.

drspangle avatar Jul 11 '21 23:07 drspangle

@drspangle cool, i can add it if you make me maintainer

leonhma avatar Jul 12 '21 05:07 leonhma

Hopefully the discussion in https://github.com/drspangle/infidel-sensor/pull/48 will help move this forwards

darigovresearch avatar Aug 08 '21 21:08 darigovresearch

@drspangle already set up the docs on RTD (you can see them here: https://infidel-sensor.readthedocs.io/, maybe add that link as the homepage). So it's set up, the firmware documentation is added with this pr and still has to be merged. It seems like @CCMCAGP noticed some unclear instructions, maybe he will help rephrase some of them later on.

Sorry if you may not hear back from me for the next week, I'm on vacation with my family and mobile network is kinda spotty here :/

leonhma avatar Aug 29 '21 20:08 leonhma

All good, thanks @leonhma for the context & no worries if you're on holiday. I would suggest then @drspangle to merge this pull request if you're happy. @CCMCAGP can make a pull request on the latest version when merged & I can update the documentation link from the readme in another pull request.

@drspangle does this work for you?

darigovresearch avatar Aug 29 '21 21:08 darigovresearch

Link to the docs has been updated via #61 . @drspangle can you advise on the above?

darigovresearch avatar Nov 20 '21 20:11 darigovresearch

I see the link is updated, I guess I've got to edit the URL in the repo configuration?

On Sat, Nov 20, 2021, 3:15 PM Darigov Research @.***> wrote:

Link to the docs has been updated via #61 https://github.com/drspangle/infidel-sensor/pull/61 . @drspangle https://github.com/drspangle can you advise on the above?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/drspangle/infidel-sensor/pull/54#issuecomment-974705877, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAU4ISZ6KPKE6BPRVK464BDUM76W5ANCNFSM5AFFSJ2A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

drspangle avatar Nov 21 '21 20:11 drspangle