Skip to content

Conversation

@rubenmoral
Copy link
Member

The example is located in the 'configuration' category.

https://jira.digi.com/browse/XBJAPI-359

Signed-off-by: Ruben Moral Ruben.Moral@digi.com

@rubenmoral rubenmoral added this to the wifi milestone Dec 13, 2016
@rubenmoral rubenmoral self-assigned this Dec 13, 2016
@rubenmoral rubenmoral requested a review from diescalo December 13, 2016 15:05
@rubenmoral rubenmoral force-pushed the rmoral/connect_ap_sample branch from cbcb568 to 6c7a991 Compare December 13, 2016 16:45
* com.digi.xbee.api.connecttoaccesspoint.MainApp.java:
Main application class. It instantiates a Wi-Fi device, establishes a
serial connection with it, connects to the specified access point and
prints some addressing information.
Copy link
Member

Choose a reason for hiding this comment

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

Replace Tabs by spaces. There are more along the document I think.

that the following message is printed out in the console of the launched
application:

">> Successfully connected to <SSID>"
Copy link
Member

Choose a reason for hiding this comment

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

Replace by:

">> Successfully connected to '<SSID>'"

(Notice the single quotes)

return;
}

myDevice.setIPAddressingMode(IPAddressingMode.DHCP);
Copy link
Member

Choose a reason for hiding this comment

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

I would configure the IP addressing mode before connecting to an access point. At least in the documentation we suggest to do it first.

The example is located in the 'configuration' category.

https://jira.digi.com/browse/XBJAPI-359

Signed-off-by: Ruben Moral <Ruben.Moral@digi.com>
@rubenmoral rubenmoral force-pushed the rmoral/connect_ap_sample branch from 6c7a991 to e44b5b0 Compare December 14, 2016 09:31
@rubenmoral rubenmoral merged commit e44b5b0 into wlan_support Dec 14, 2016
@rubenmoral rubenmoral deleted the rmoral/connect_ap_sample branch December 14, 2016 09:34
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