-
Notifications
You must be signed in to change notification settings - Fork 876
Frontend Workspace Resources Card #1031
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
Comments
There is a meeting on the calendar to discuss this one for details. |
Hey team! Please add your planning poker estimate with ZenHub @presleyp @kylecarbs |
table of resources with ability to open each in web terminal Button on each row called Open In Terminal |
persistent/ephemeral is a different type of data than connection status. can call that column Lifetime? |
subrows: could make it monospace and use the ascii elbow |
Open Port button - opens dropdown where you type port number DevURLs are predefined port allocations (and use a certain resource) But resources can be opened in any port. If you open dev in the port VSCode is assigned to, you see VSCode. |
Look into nesting dev urls/applications in resources card |
make agent connection status optional so we can reuse this component for a preview |
@kylecarbs this one needs input from you in a convo with @presleyp so that she has all the right information to implement this. Can you list the endpoints that she will need? This may also be something that we could use your help with for community since it is pretty large. Maybe the two of you can separate the endpoint hook ups with the views. |
@vapurrmaid To help groom this and break it apart with Presley so we can better spread the load on it. |
Breakout session notes Lines 11 to 39 in 982769f
coder/codersdk/workspacebuilds.go Lines 61 to 72 in 982769f
The API:
coder/codersdk/workspaceresources.go Lines 23 to 31 in 537897c
Dev URLs will be added to this later (same endpoint). Desired (follow-up): open resource in terminal. We need to create another ticket for that. Step 1 Present the data similarly to the CLI in a table. Step 2 We're going to clean up the UI (TBD) Step 3 Dev URLs Job associated to WorkspaceResource (job created resource) Workspace and WorkspaceResource do not have a relationship (job creates the resource) Request --> We can safely poll without 4xx or 5xx, because the data is static (not actually touching the workspace itself). The FE can remain open and continue to poll. |
"Desired (follow-up): open resource in terminal. We need to create another ticket for that." @code-asher isn't this essentially #760? It's just that instead of having one terminal link like in v1 there's one per workspace agent as shown here. |
And adding Dev URLs is #805 |
Yup I think #760 is perfect for this |
We're currently blocked on this (@deansheather 's working on something) |
lifecycle is computed on the frontend - we get what resources exist in Stop and what exist in Start, and those that exist in Start but not Stop, they're ephemeral (gets destroyed on Stop). query the template version for its resources. can't query workspace build bc it only shows for the state it's currently in. will get a flat array where each item is a resource with a workspace transition, so need to look for the resource by its name and type and see if the array has one with stop and one with start. prob a nice to have. |
Agents will have an array of Apps soon, and status. WorkspaceApp will have external URL and I just need to render name, icon, and a link to that url. |
@BrunoQuaresma this is an example of what would go above the timeline in #1620 |
Ok. How are we going to add DEV URLs and apps inside of this? |
What do you think about removing the |
@BrunoQuaresma that's what I meant by "I'm going to take out Lifecycle and Access" - I just put them in for the screenshot so we could all be on the same page about how it will look eventually. DevURLs and apps will go in the Access column. |
Handled in other tickets:
The text was updated successfully, but these errors were encountered: