Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Proposal: consolidate parameter memory #14757

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

Closed
ammario opened this issue Sep 20, 2024 · 2 comments
Closed

Proposal: consolidate parameter memory #14757

ammario opened this issue Sep 20, 2024 · 2 comments
Assignees

Comments

@ammario
Copy link
Member

ammario commented Sep 20, 2024

When discussing user-level parameters with @stirby I came to an idea about how we can simplify the parameter system. Right now we have an ephemeral bool flag on coder_parameter and our plan of action for introducing autofill was to create an autofill bool flag. I realize these features have conceptual overlap and would be better represented a single flag on parameters:

memory = "build" | "workspace" | "template" | "org" | "user"

This option would default to workspace. memory = user solves for user-level parameters. memory = build is equivalent to today's ephemeral = true.

We would need to hide, deprecate, and remove the ephemeral opt—which is a happy thing for me.

Related:

cc @code-asher

@stirby stirby added feature roadmap https://coder.com/roadmap. Only humans may set this. roadmap-maybe Ideas we're considering! Only humans may set this. labels Sep 27, 2024
@matifali matifali removed the feature label Oct 14, 2024
@stirby
Copy link
Collaborator

stirby commented Oct 16, 2024

We're doing this as part of "Magic Parameters."

@stirby stirby removed the roadmap-maybe Ideas we're considering! Only humans may set this. label Oct 25, 2024
@stirby stirby mentioned this issue Oct 25, 2024
8 tasks
@stirby stirby removed the roadmap https://coder.com/roadmap. Only humans may set this. label Oct 28, 2024
@bpmct
Copy link
Member

bpmct commented Nov 21, 2024

We're working on a new design

@bpmct bpmct closed this as not planned Won't fix, can't repro, duplicate, stale Nov 21, 2024
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

No branches or pull requests

5 participants