Skip to content

Conversation

@Tymek
Copy link
Contributor

@Tymek Tymek commented May 21, 2025

DO NOT MERGE

@vercel
Copy link

vercel bot commented May 21, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
unleash-docs ⬜️ Ignored (Inspect) Visit Preview May 21, 2025 5:34pm

@github-actions
Copy link
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@ivarconr ivarconr moved this from New to In Progress in Issues and PRs May 22, 2025
Copy link
Contributor

@gastonfournier gastonfournier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good but I'd try to split this into different PRs

* Run `yarn gen:api` to generate the client.
* We may use methods (src/openapi/apis) for new features in the future.
*/
module.exports = {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel this should be a separate PR fixing orval config.

Comment on lines +101 to +105
{rows.length > 0 ? (
<Box sx={{ mb: 4 }}>
<ApiTokenDocs />
</Box>
) : null}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This refactoring makes sense but I'd do it as a different PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

3 participants