daytona icon indicating copy to clipboard operation
daytona copied to clipboard

Improve info command

Open ezhil56x opened this issue 11 months ago • 6 comments

Improve info command

Description

  • This PR is to improve daytona info command in cli
  • Added support to display WorkspaceId, Target, RepositoryUrl

Fixes #114 /claim #114

Type of Change

  • [x] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] This change requires a documentation update

Checklist

  • [x] I have performed a self-review of my own code
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [ ] My changes generate no new warnings

Related Issue(s)

This PR addresses issue #114

Screenshots

image

ezhil56x avatar Mar 08 '24 08:03 ezhil56x

Thank you for the PR @ezhil56x

This looks good - just a couple of changes: As you can see in the screenshot of a multi project workspace below, we should move the “repository url” to the projects section between each project name and state line. While you/re at it, also rename the wokrspaceURLStyle to repositoryURLStyle since the repository is the one that has the URL.

Last thing we could do is change the label WorkspaceId to just “ID” for less clutter.

Additionally, if you didn't already, please read through our contributing guidelines. Specifically, this section which requires you to add your username to the CONTRIBUTORS.md file.

Screenshot 2024-03-08 at 10 27 54

idagelic avatar Mar 08 '24 09:03 idagelic

@ezhil56x By the way, here is an example of adding your entry to CONTRIBUTORS.md - a separate PR and specific commit message: https://github.com/daytonaio/daytona/commit/2f3e8b9ac3ce1bdf3f379ab3ad9f2951cd02fb41

idagelic avatar Mar 08 '24 10:03 idagelic

@idagelic Added to CONTRIBUTORS.md and made required changes. Can you review once?

Screenshot after changes

image

ezhil56x avatar Mar 08 '24 11:03 ezhil56x

Thanks @ezhil56x - last thing I would ask you to do is to remove the contribution commit from this PR and add it to another one just like this https://github.com/daytonaio/daytona/commit/2f3e8b9ac3ce1bdf3f379ab3ad9f2951cd02fb41 - with a commit message chore: add "username" to contributors and submit it as well.

idagelic avatar Mar 08 '24 12:03 idagelic

@idagelic Done 👍🏻

ezhil56x avatar Mar 08 '24 12:03 ezhil56x

@idagelic @Tpuljak

Required changes are done image

ezhil56x avatar Mar 08 '24 17:03 ezhil56x