-
Notifications
You must be signed in to change notification settings - Fork 61
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
Improve "index" handling #202
Comments
As also indicated in #201, there are already implementation that treat the index as an server generated immutable, project unique number to identify topics. BIMcollab uses the index this way. We could generate a user friendly Id conform #201 from this number, but for compatibility with existing implementations, I would propose not to change this property but instead add another way of defining a sort order. Changing any issue property would also change it's modification timestamp and I would not want a sorting order change to change all may topics. Thus for user defined sorting maybe an additional endpoint is needed that would only carry a list of issue identifiers in the sort order wanted. This would also be much faster on projects with lots of topics as otherwise each individual topic would have to be modified |
summary of the discussion today:
|
See here for the BCF API issue: buildingSMART/BCF-API#202
Add note stating that index property is deprecated
Closed by #244 |
Hi guys, I know this issue is closed, but I'd like to raise the fact that by replacing an integer / float by a string will most likely introduce some sorting issues. Moreover, in previous versions, the client could specify the index, and now it has become a server ID. Relying solely on the server will introduce some strange sorting behaviors in situations where the client would try to simultaneously perform some POST requests. In this case, the sorting of the issues can be altered and cannot be "controlled" by the client. What will be the expected behavior in this case? |
After communication with Solibri, we see the need to improve how "index" is specified in BCF-API documentation.
Would be nice to discuss this at todays telecon
The text was updated successfully, but these errors were encountered: