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

ElementID output for ElemDetails node #216

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tlorantfy
Copy link
Collaborator

Solved issue: #215

Screenshot 2025-01-12 104140

@tlorantfy
Copy link
Collaborator Author

@JBENEITEZ , could you please review the name of the output parameters of the ElemDetails node?

It already had ElementIds output (which is the list of GUIDs), so I chose to name the new output ElementIDPropValues.
Do you have better idea?

@SzamosiMate
Copy link

When you say 'Element ID' Archicad users will think of the built-in property.
I think it was pretty confuseing from Graphisoft's part to use this name for a GUID/UUID container.

I would use Element ID for the built-in property, and go for a new name for the GUID/UUID container.
What about ElementGUID? It is more specific. Or maybe Element? It is short, and from the user's perspective it is used to represent elements.

@JBENEITEZ
Copy link
Contributor

JBENEITEZ commented Jan 13, 2025 via email

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.

3 participants