Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: Use correct JSON-LD context prefix #257

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

leandro-cavalcante
Copy link
Contributor

Description

In the returned policies, the policy hub adds the https://w3id.org/catenax/v0.0.1/ns/ context with the cx prefix, but then uses the cx-policy prefix instead.

Why

The cx-policy prefix isn’t bound to any context within the JSON-LD document and therefore will fail to expand during serialization.

Issue

N/A

Issue

Link to Github issue.

Checklist

Please delete options that are not relevant.

  • I have followed the contributing guidelines
  • I have performed IP checks for added or updated 3rd party libraries
  • I have created and linked IP issues or requested their creation by a committer
  • I have performed a self-review of my own code
  • I have successfully tested my changes locally
  • I have added tests that prove my changes work
  • I have checked that new and existing tests pass locally with my changes
  • I have commented my code, particularly in hard-to-understand areas
  • I have added copyright and license headers, footers (for .md files) or files (for images)

@leandro-cavalcante
Copy link
Contributor Author

@evegufy the PR #232 was closed due some issues I had within my branch. I opened this one and it already has the changes requested by @Phil91

@evegufy evegufy requested review from Phil91 and evegufy February 7, 2025 08:09
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

1 participant