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

[HuaXiang] 10309 #76

Merged
merged 9 commits into from
Apr 1, 2019

Conversation

JiaDeng-Git
Copy link
Contributor

new pull request template

1.We deleted the "Bluetooth" resource,because it isn't the best way  to use a specific Wireless chip.
2.We deleted the "LockState" resource,because we can deduce it from other factors.
2.For different application scenarios,we added the "LockType" resource .
3.We changed the "LightState" to "LightSwitchState".
4.For Knowing the device of Wireless state,we added the "RSSI" resource. 
5.We changed the "UpTime" to "DataUpTime" and the resource's String types to Time.
6.And we added two new resources variables,which are "Latitude" and "Longitude". "Latitude",which ID is 5514. "Longitude",which ID is 5515.
@jpradocueva jpradocueva changed the title 10309 [HuaXiang] 10309 Feb 28, 2019
update file name from ShareParkingLock to 10309.
@jpradocueva
Copy link
Member

jpradocueva commented Feb 28, 2019

@dengjialean
We need to resolve the following issues:

  • Is "ShareParkingLock" a replacement of "DeviceDataContainer"?
  • Software license is too restricted. By setting the copyright to Hua Xiang all rights reserved, other companies won't be able to use this Object. Are you considering to have a more open software license?
  • Incorrect schema filename: LWM2M-v1_0.xsd, should be name LWM2M.xsd.
  • LwM2M schema is set to v1.0 but the "LWM2MVersion" of the protocol is set to "1.1". This is incorrect. You need to decide if this Object is release for version 1.0 or v1.1 of LwM2M protocol.
  • URN is set to urn:oma:lwm2m:x:10309:1.0, as this object is the first version (1.0), there is no need to add the Object version to the URN. Therefore, URN needs to be changed to: urn:oma:lwm2m:x:10309
  • Once the above issues are addressed, we will validate the Object

1.We decided to change the name of object 10309 "DeviceDataContainer" to "ShareParkingLock".
2.We think this object should be public.
@JiaDeng-Git
Copy link
Contributor Author

JiaDeng-Git commented Mar 3, 2019 via email

@ScottTaftPotter
Copy link
Contributor

We advise you to use functionality from existing objects instead of creating a new object. For instance, please consider the following:

  • Location (object number 6) for latitude and longitude
  • Device (object number 3) for battery level
  • Connectivity Monitoring (object number 4) for RSSI

@JiaDeng-Git
Copy link
Contributor Author

We advise you to use functionality from existing objects instead of creating a new object. For instance, please consider the following:

  • Location (object number 6) for latitude and longitude
  • Device (object number 3) for battery level
  • Connectivity Monitoring (object number 4) for RSSI

Editorial changes to range an enumeration and descriptions.
@ScottTaftPotter
Copy link
Contributor

Using the decomposition we recommended would have preferable, but given your urgency, we have made only minor edits. In the future we will provide better guidelines for best practices.

@ScottTaftPotter ScottTaftPotter merged commit 3591f5b into OpenMobileAlliance:HuaXiang Apr 1, 2019
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.

None yet

3 participants