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

Add Samples to Modeling App #1350

Open
jgomez720 opened this issue Feb 6, 2024 · 3 comments · May be fixed by #3912
Open

Add Samples to Modeling App #1350

jgomez720 opened this issue Feb 6, 2024 · 3 comments · May be fixed by #3912
Assignees
Labels
enhancement New feature or request

Comments

@jgomez720
Copy link
Collaborator

jgomez720 commented Feb 6, 2024

This would give users a button in which they can access our sample KCL files (pulls directly from kcl-samples repo). When user selects the option, they get prompted with "Are you sure you want to import the sample code? This will overwrite the code you currently have"

Screenshot 2024-02-06 at 12 02 00 PM
@jgomez720 jgomez720 self-assigned this Feb 6, 2024
@jgomez720
Copy link
Collaborator Author

jgomez720 commented Feb 6, 2024

Having a direct import of the code would be great for the future, but I am fine settling with a link in the three-dot menu for now:
#1352

@franknoirot
Copy link
Collaborator

Yes, I think we could add an Import example command with the following arguments:

  1. method: will be new file by default on desktop and can only be overwrite on web
  2. example: will have a list of all the KCL samples from the repo as options. Will be used to fetch the actual code when executed

We'll hook up a confirmation dialog if the method is overwrite

@jgomez720
Copy link
Collaborator Author

Yeah this would be sick!

@franknoirot franknoirot linked a pull request Sep 18, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants