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 group interaction command #437

Closed
Cedrok opened this issue Apr 15, 2024 · 1 comment · Fixed by #446
Closed

New group interaction command #437

Cedrok opened this issue Apr 15, 2024 · 1 comment · Fixed by #446
Assignees
Labels

Comments

@Cedrok
Copy link
Member

Cedrok commented Apr 15, 2024

The command group.content=true|false send an interact command to the 3D client to display or not the content of a group. But this remove the possibility to modify the content of a group (which is now supported by the 3D client).
To fix that, we need 2 separated commands:

  • modify
    group.content=item1,item2,...,itemN => modify the object in the API (then the API send the modified object to everyone)

  • interact
    group.displayContent=true|false => Replace the "old" content command for the 3D

@Cedrok Cedrok added enhancement New feature or request CLI Priority: Medium labels Apr 15, 2024
@helderbetiol
Copy link
Collaborator

This has been implemented with the typed attributes #443. Here's an example of how it works now:

admin@TenantName:/Physical/BASIC/A/R1 [TenantName]> GR1:content=[A07, A08,A09]
admin@TenantName:/Physical/BASIC/A/R1 [TenantName]> get GR1
{
    "attributes": {
        "content": ["A07","A08","A09"]
    }
    "category": "group"
    "createdDate": "2024-04-18T14:48:44.558Z"
    "description": ""
    "domain": "TenantName"
    "id": "BASIC.A.R1.GR1"
    "lastUpdated": "2024-04-24T09:57:12.907Z"
    "name": "GR1"
    "parentId": "BASIC.A.R1"
    "tags": []
}
admin@TenantName:/Physical/BASIC/A/R1 [TenantName]> GR1:content=[A01..A08]
admin@TenantName:/Physical/BASIC/A/R1 [TenantName]> get GR1
{
    "attributes": {
        "content": ["A01","A02","A03","A04","A05","A06","A07","A08"]
    }
    "category": "group"
    "createdDate": "2024-04-18T14:48:44.558Z"
    "description": ""
    "domain": "TenantName"
    "id": "BASIC.A.R1.GR1"
    "lastUpdated": "2024-04-24T09:57:52.134Z"
    "name": "GR1"
    "parentId": "BASIC.A.R1"
    "tags": []
}
admin@TenantName:/Physical/BASIC/A/R1 [TenantName]> GR1:displayContent=false
admin@TenantName:/Physical/BASIC/A/R1 [TenantName]> GR1:displayContent=falsd
Error: displayContent should be a boolean
admin@TenantName:/Physical/BASIC/A/R1 [TenantName]> GR1:displayContent=true

@helderbetiol helderbetiol self-assigned this Apr 24, 2024
@helderbetiol helderbetiol linked a pull request Apr 24, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants