Skip to content

Add documentation of MDNS #197

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

Merged
merged 1 commit into from
Jan 7, 2020
Merged

Add documentation of MDNS #197

merged 1 commit into from
Jan 7, 2020

Conversation

husigeza
Copy link
Contributor

MDNS feature is released with PR: pycom/pycom-micropython-sigfox#377
Adding documentation for it.

@husigeza husigeza requested review from jirkadev and lilycey December 12, 2019 16:52
Copy link
Contributor

@lilycey lilycey left a comment

Choose a reason for hiding this comment

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

All good bar one small question!


The arguments are:

* `service_type` is the type of the offered service, e.g.: _http, _ftp or can be custom service
Copy link
Contributor

Choose a reason for hiding this comment

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

The formatting here seems slightly different - will it upload well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I checked and the content of the mdns.md is exactly the same as what we have on the publish-development branch. Is that a problem?

@jirkadev
Copy link
Contributor

jirkadev commented Dec 13, 2019

If this is the stable firmware feature, we should open the second PR and merge this branch also to development-publish

I wrote a script which can open two PRs automatically and assign a reviewer to it here
https://github.com/pycom/pycom-documentation/tree/publish/scripts

@husigeza
Copy link
Contributor Author

The documentation of this feature has already been merged into development-publish: #183

@jirkadev jirkadev merged commit dbbca99 into pycom:publish Jan 7, 2020
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.

3 participants