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

New field "type" in json generated by LwM2mIdentitySerDes and LwM2mPeerSerDes. #1459

Conversation

JaroslawLegierski
Copy link
Contributor

Implementation of field "type" in json generated by LwM2mIdentitySerDes and LwM2mPeerSerDes.

@JaroslawLegierski JaroslawLegierski force-pushed the opl/type_field_in_json_LwM2mIdentitySerDes_LwM2mPeerSerDes branch from aae4289 to 282ec58 Compare June 20, 2023 16:11
@sbernard31
Copy link
Contributor

sbernard31 commented Jun 26, 2023

I Integrated this in identity_refactoring branch. (squashed in one commit 26a050a)

@JaroslawLegierski there is a way to launch build to test Redis stores. Personally I use : mvn clean install javadoc:javadoc -Predis

The redis profile can only be launched locally with a redis server running on localhost by default (it could be changed using REDIS_URI environnement variable). This is not done by jenkins OR github action because we have no redis instance available.

When I did it, RedisRegistrationTest failed ❌.
With this fix that sounds OK ✔️. (This fix in integrated in squashed commit)

I guess we can close this PR now.

@sbernard31 sbernard31 closed this Jun 26, 2023
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