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

Make "Extent" entries in items a reference #14

Open
ribose-jeffreylau opened this issue Mar 23, 2023 · 7 comments
Open

Make "Extent" entries in items a reference #14

ribose-jeffreylau opened this issue Mar 23, 2023 · 7 comments
Assignees
Labels
enhancement New feature or request

Comments

@ribose-jeffreylau
Copy link
Contributor

... so that they can be reused by different items.

Updating an Extent should automatically update it for all items referencing it.

from discussion with @maccraymer

@strogonoff
Copy link
Contributor

strogonoff commented Mar 23, 2023

I think we cannot make it a reference because we cannot introduce a register item class without going through control body register model/data model change approval. What we can do is a feature that allows to import an extent from another item, similar to how it works in Java backend.

@maccraymer
Copy link

maccraymer commented Mar 23, 2023 via email

@maccraymer
Copy link

maccraymer commented Mar 23, 2023 via email

@ronaldtse ronaldtse transferred this issue from paneron/registry-kit Mar 23, 2023
@ronaldtse
Copy link

@maccraymer is correct that we should make the Extent a register item class (or concept class in the new 19135 terminology). Since Extents are often reused, we should manage them as register items.

@strogonoff
Copy link
Contributor

strogonoff commented Mar 24, 2023

@maccraymer is correct that we should make the Extent a register item class (or concept class in the new 19135 terminology). Since Extents are often reused, we should manage them as register items.

@ronaldtse From our meeting last Monday I got a strong impression that we are not to make unilateral changes to register data model. Should we submit the addition of a new register item class “Extent” for consideration to control body?

@ronaldtse
Copy link

@maccraymer is the chair of the control body; this topic has been discussed extensively at the control body. However I do not know if we have had a formal "control body decision"?

@strogonoff
Copy link
Contributor

For now extent import had been implemented for feature parity. This was discussed on the call first (but the issue was not filed, perhaps that point slipped through the cracks) as an important feature while extents are not separate items and cannot be referenced.

Screenshot 2023-03-26 at 03 05 11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants