Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat: Caution section on code generation #42

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

Conversation

mobley-trent
Copy link
Contributor

Closes #19
/claim #19

@mobley-trent
Copy link
Contributor Author

Hello @nkkko I think this should fix how the caution section renders.

@mojafa
Copy link
Contributor

mojafa commented Dec 7, 2024

@mobley-trent I'm on it

@mobley-trent
Copy link
Contributor Author

hi @mojafa is the PR satisfactory ?

@mojafa
Copy link
Contributor

mojafa commented Dec 11, 2024

@mobley-trent I’ve updated main with some changes that affect this PR. Could you please rebase your branch onto the latest main and resolve any conflicts that come up? Once that’s done, we can review the updated changes and move forward.

@mobley-trent
Copy link
Contributor Author

@mojafa I've rebased the branch on top of upstream/main

@mojafa
Copy link
Contributor

mojafa commented Dec 18, 2024

@mobley-trent please share a screenshot of your solution.

@mojafa
Copy link
Contributor

mojafa commented Dec 18, 2024

or screen recording

Signed-off-by: Eddy Oyieko <oyieko99@gmail.com>
Signed-off-by: Eddy Oyieko <oyieko99@gmail.com>
@mobley-trent
Copy link
Contributor Author

@mojafa I added the caution section to the error div as well so that you can see how it looks.

Screenshot 2024-12-19 111931

Signed-off-by: Eddy Oyieko <oyieko99@gmail.com>
@mojafa
Copy link
Contributor

mojafa commented Dec 19, 2024

@mobley-trent good one. It works now.

Just a slight request, please put the warning display before the generated devcontainer?
Screenshot 2024-12-19 at 13 06 50

Signed-off-by: Eddy Oyieko <oyieko99@gmail.com>
Signed-off-by: Eddy Oyieko <oyieko99@gmail.com>
@mobley-trent
Copy link
Contributor Author

@mojafa done

Signed-off-by: Eddy Oyieko <oyieko99@gmail.com>
@mojafa
Copy link
Contributor

mojafa commented Dec 21, 2024

@nkkko please review this. Meets the requirements of the issue

Screenshot 2024-12-21 at 07 05 21

Signed-off-by: Eddy Oyieko <oyieko99@gmail.com>
@mobley-trent
Copy link
Contributor Author

added a margin between the caution div and code section

@mojafa
Copy link
Contributor

mojafa commented Dec 21, 2024

@mobley-trent thanks.

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.

Add Note to Caution Users About Executing AI-Generated Code
2 participants