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

Add deprecation message to wifi.sta.config() for argument style cfg #1939

Merged

Conversation

dnc40085
Copy link
Contributor

@dnc40085 dnc40085 commented Apr 26, 2017

  • This PR is for the dev branch rather than for master.
  • This PR is compliant with the other contributing guidelines as well (if not, please describe why).
  • I have thoroughly tested my contribution.

The documentation for the old(argument) style of station configuration was removed in a previous PR, but at the time the function platform_print_deprecation_note() did not exist, so I wasn't able to add a message informing of the removal.

Now that the function exists, I decided it was time to add a deprecation message to inform the developer of the upcoming removal the argument style of station configuration.

Documentation for the old(argument) style of station configuration was
removed in a previous PR, so I added a deprecation message to inform
developer of the upcoming removal of the feature.
@dnc40085 dnc40085 changed the title Added deprecation message to wifi.sta.config() for argument style cfg Add deprecation message to wifi.sta.config() for argument style cfg Apr 26, 2017
@marcelstoer marcelstoer added this to the 2.0.0-follow-up milestone Apr 26, 2017
@marcelstoer marcelstoer merged commit 3e24b1c into nodemcu:dev Apr 26, 2017
@marcelstoer
Copy link
Member

Thanks for being so diligent.

@dnc40085 dnc40085 deleted the dev_add_deprecation_msg_to_wifi.sta.config branch April 26, 2017 13:01
bLandais added a commit to bLandais/constellation-documentation that referenced this pull request Nov 6, 2017
Deux modifications dans la partie du code Wifi Lua : 

1. La fonction wifi.sta.config() est dépréciée dans le firmware 2.1 (voir nodemcu/nodemcu-firmware#1939 )
2. Il manquait un end à la fin de la fonction, ce qui bloquait la compilation
eiselekd pushed a commit to eiselekd/nodemcu-firmware that referenced this pull request Jan 7, 2018
…nodemcu#1939)

Documentation for the old(argument) style of station configuration was
removed in a previous PR, so I added a deprecation message to inform
developer of the upcoming removal of the feature.
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