-
Notifications
You must be signed in to change notification settings - Fork 6
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
Do not insert all oid prefixes on db creation level #333
Conversation
It is not a good practice to store the oid prefixes in the main datamodel. The sysadmin should add their own prefixes to their datbase instead
I feel we should ease as much the configuration of things at least for TEKSI users and why not preconfigure oid prefixes? |
It is ok to have some fake oids for testing purposes, but I do not see an advantage in centrally storing all prefixes |
I think it is better to have a GUI allowing to set that value instead. You'll have to do it once and nobody else needs that information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok for me to take this out here. With more than 40 members and xxx communities this would become too big and difficult to maintain.
If we want to have INTERLIS checks in the future, we have at least to have an invalid and a valid prefix in the sample data. |
@ponceta we can use the prefix |
@ponceta are you ok with merging this PR? Your comment is separate from it |
General
Context
It is not a good practice to store the oid prefixes in the main datamodel. The sysadmin should add their own prefixes to their datbase instead
Describe your changes
Remove provided oid prefixes
Screenshots
Issue ticket number and link
Checklist before requesting a review
Checklist before merge