Skip to content

Conversation

@msorens
Copy link
Contributor

@msorens msorens commented Sep 7, 2025

Why the changes in this PR are needed?

Example on the OPA home page in the AI category was broken.

Previously output of default rego was this, i.e. the available models list was empty:

[
  "Model 'model-2' is not in your accessible models: "
]

Output of default rego is now this:

[
  "Model 'model-2' is not in your accessible models: ^model-\\d+-stage$, model-1"
]

What are the changes in this PR?

  • Corrected path to the input element.
  • PR bonus: added explanatory text to aid new users.

Notes to assist PR review:

NA

Further comments:

NA

Previously output of default rego was this, i.e. the available models list was empty.
```
[
  "Model 'model-2' is not in your accessible models: "
]
```

Output of default rego is now this:
```
[
  "Model 'model-2' is not in your accessible models: ^model-\\d+-stage$, model-1"
]
```

Signed-off-by: Michael Sorens <[email protected]>
@msorens msorens changed the title Msorens/example fix home page example fix Sep 7, 2025
@netlify
Copy link

netlify bot commented Sep 7, 2025

Deploy Preview for openpolicyagent ready!

Name Link
🔨 Latest commit 54df9e8
🔍 Latest deploy log https://app.netlify.com/projects/openpolicyagent/deploys/68bcd61a5863da0008db226c
😎 Deploy Preview https://deploy-preview-7885--openpolicyagent.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@anderseknert anderseknert merged commit cbaf314 into open-policy-agent:main Sep 7, 2025
20 checks passed
@msorens msorens deleted the msorens/example-fix branch September 8, 2025 00:20
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.

2 participants