Skip to content

Comments

Add solution#6

Merged
IvanKiral merged 6 commits intomainfrom
add_solution
Mar 31, 2025
Merged

Add solution#6
IvanKiral merged 6 commits intomainfrom
add_solution

Conversation

@IvanKiral
Copy link
Contributor

Motivation

Which issue does this fix? Fixes #issue number

If no issue exists, what is the fix or new feature? Were there any reasons to fix/implement things that are not obvious?

Checklist

  • Code follows coding conventions held in this repo
  • Automated tests have been added
  • Tests are passing
  • Docs have been updated (if applicable)
  • Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test

If manual testing is required, what are the steps?

@vercel
Copy link

vercel bot commented Feb 27, 2025

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

Name Status Preview Comments Updated (UTC)
kickstart ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 18, 2025 7:48am
kickstart-qa ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 18, 2025 7:48am

README.md Outdated
npm run model:generate
```
> [!NOTE]
> The `VITE_MANAGEMENT_API_KEY` variable inside .env file must be set to generate models properly.
Copy link
Member

Choose a reason for hiding this comment

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

And VITE_ENVIRONMENT_ID as well it seems.

const SolutionList: React.FC = () => {
const { environmentId, apiKey } = useAppContext();

const [solutions, setSolutions] = useState<Solution[] | null>(null);
Copy link
Member

Choose a reason for hiding this comment

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

The array type should be readonly as it is forbidden to modify anything inside a react state.

*
* Project: Kontent.ai Kickstart
* Environment: Production
* Id: 133f5ef8-66b1-0074-7d42-55371447aac4
Copy link
Member

Choose a reason for hiding this comment

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

Can we have it without the ids? I don't think they are helpful.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It helped me once to figure out, that the provided environment dindn't had good relations set 🗡️

Other thing is I tried but new model generator doesn't have the option :/ only if i generate it to the memory and remove it with regex

externalId: undefined,
name: "Workshop",
codename: "workshop",
id: "707bb142-f7d5-574a-b0ba-63037c412d84",
Copy link
Member

Choose a reason for hiding this comment

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

Same here, the ids are gonna be different for every project.

@IvanKiral IvanKiral merged commit 8f1f20e into main Mar 31, 2025
4 checks passed
@IvanKiral IvanKiral deleted the add_solution branch August 25, 2025 14:08
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