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

Update to Firmware 7.25 #18

Merged
merged 11 commits into from
Mar 3, 2021
Merged

Conversation

SmokingDevices
Copy link
Contributor

Update to latest firmware version.

Copy link
Owner

@kaklakariada kaklakariada left a comment

Choose a reason for hiding this comment

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

Thanks for your pull request! Please fix my findings, then we can merge this.

private int last;

@Text
private String name;

public boolean isLast() {
Copy link
Owner

Choose a reason for hiding this comment

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

Sorry for nit-picking. But if this returns an int, we should rename the getter.

Suggested change
public boolean isLast() {
public boolean getLast() {

Copy link
Contributor Author

@SmokingDevices SmokingDevices Feb 28, 2021

Choose a reason for hiding this comment

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

According to the documentation (Page 2: "Bedeutung der Werte:") this should indicate the last user which then (from my point of view) is a boolean.

My poor XML skills didn't bring me to a solution where i could mark the field to be a boolean and the xml annotation would do the translation from 1 to true.

Copy link
Owner

Choose a reason for hiding this comment

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

👍

@kaklakariada
Copy link
Owner

Please also add an entry to the Changelog. Previously i always did this myself, but you know better how to describe your changes :)

}

@Test
public void parseSessionInfo() throws IOException {
Copy link
Contributor

Choose a reason for hiding this comment

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

[Minor] With my from today PR I just introduced assertJ into the project. Maybe you want to have a look at those Assertion classes which allow fluent-API assertions. BR, Peter.

Copy link
Contributor Author

@SmokingDevices SmokingDevices Feb 28, 2021

Choose a reason for hiding this comment

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

Will have a look, but as rebase is not my favourite git command, i would update these tests with my next contribution.

Copy link
Owner

Choose a reason for hiding this comment

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

Rebasing ist not required. I also don't use it ;)

@@ -7,7 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
## [1.4.0] - unreleased

### Added

* [#18] Added new fields to be compatible with Fritz!OS 7.25
Copy link
Owner

Choose a reason for hiding this comment

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

👍

private int last;

@Text
private String name;

public boolean isLast() {
Copy link
Owner

Choose a reason for hiding this comment

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

👍

@kaklakariada kaklakariada merged commit ce1b8de into kaklakariada:master Mar 3, 2021
@kaklakariada
Copy link
Owner

I just created a new release 1.4.0 and published it to Maven Central. It may take a while until it is available at https://repo1.maven.org/maven2/com/github/kaklakariada/fritzbox-java-api/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants