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

adding featureId to ldap schema #90

Merged
merged 3 commits into from
Mar 3, 2025
Merged

adding featureId to ldap schema #90

merged 3 commits into from
Mar 3, 2025

Conversation

RemiDesgrange
Copy link
Contributor

No description provided.

@fvanderbiest
Copy link
Member

@RemiDesgrange can you refresh my memory - why was this contributed originally ?
What's the status of this PR ?

@pmauduit
Copy link
Member

pmauduit commented Feb 28, 2025

1.2.2 is not "featureId" anymore, relying on the current state of the geOrchestra LDIF schema, it should be "georchestraObjectIdentifier" and should match a UUID: https://oid-base.com/get/1.3.6.1.1.16.1, cf:
https://github.com/georchestra/georchestra/blob/master/ldap/docker-root/georchestraSchema.ldif#L16-L20

And I am wondering: Shall we have a documentation for every attributes being used in the georchestra schema, or just pointing to the LDIF in the source tree should be considered sufficient as documentation ?

@pmauduit
Copy link
Member

pmauduit commented Mar 3, 2025

updated current branch, see last commit.

Rémi Desgrange and others added 3 commits March 3, 2025 12:51
@pmauduit
Copy link
Member

pmauduit commented Mar 3, 2025

rebased on master (hence the push-force).

@pmauduit
Copy link
Member

pmauduit commented Mar 3, 2025

GHA is failing, but this does not look like related to this PR, some errors are about the X/twitter account.

@pmauduit pmauduit merged commit 3b00f1f into master Mar 3, 2025
1 check failed
@pmauduit pmauduit deleted the update-oid branch March 3, 2025 11:58
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