FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Feat(Core): Implement HL API by stonebuzz · Pull Request #1122 · pluginsGLPI/fields · GitHub

Feat(Core): Implement HL API - #1122

Draft
stonebuzz wants to merge 1 commit into
mainfrom
feat_hlapi
Draft

Feat(Core): Implement HL API#1122
stonebuzz wants to merge 1 commit into
mainfrom
feat_hlapi

Conversation

Copy link
Copy Markdown
Contributor

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

Replace #670

Screenshots (if appropriate):

stonebuzz marked this pull request as draft January 29, 2026 08:27
stonebuzz self-assigned this Jan 29, 2026

Copy link
Copy Markdown
Contributor Author

@SkiAn2007

Can you try this ?

Best regards

Copy link
Copy Markdown
Contributor Author

@cconard96

I'd like your opinion on this.

Thank you in advance

cconard96 commented Jan 29, 2026
edited
Loading

Copy link
Copy Markdown
Member

I don't have the plugin installed/configured anymore, but I didn't see anything in the code itself of concern as long as there is no way for the fields plugin to add fields to native custom assets and custom dropdowns. Both add custom fields to a "custom_fields" property and it doesn't seem like this code checks if it already exists to merge the properties instead.

Edit: Well maybe not... I haven't fully woken up yet. Are you trying to add the custom fields into the existing asset schemas or create new ones?

stonebuzz commented Jan 29, 2026
edited
Loading

Copy link
Copy Markdown
Contributor Author

Yes, now fields can add container fields to custom assets (not custom dropdown).

cf : #1112

smurdza commented Jun 12, 2026

Copy link
Copy Markdown

@stonebuzz do you know when this implement will be ready ?

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL