Skip to content

Commit d2b6697

Browse files
authored
Consistent UX across all access points in a create Dev Env panel (#3571)
* Consistent UX in all entry points to create dev env * Fixed InstanceType issue in workspaceGroup dev env panel * Addressed PR comments * detekt issues * Resolved PR comments * Added changLog
1 parent 3ef1180 commit d2b6697

File tree

7 files changed

+215
-246
lines changed

7 files changed

+215
-246
lines changed
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
{
2+
"type" : "bugfix",
3+
"description" : "Provided UX consistent in all access points for creating Dev Environment"
4+
}

jetbrains-core/src-222+/software/aws/toolkits/jetbrains/services/caws/CawsParameterDescriptions.kt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -48,11 +48,11 @@ fun isSubscriptionFreeTier(
4848
client: CodeCatalystClient,
4949
space: String?
5050
): Boolean {
51-
val subscriptionTier = if (existingProject != null) {
52-
checkIfSubscriptionIsFreeTier(client, existingProject.space)
51+
val subscriptionTier = if (space != null) {
52+
checkIfSubscriptionIsFreeTier(client, space)
5353
} else {
54-
space?.let { checkIfSubscriptionIsFreeTier(client, it) }
55-
} ?: return true
54+
return true
55+
}
5656

5757
return subscriptionTier == "FREE"
5858
}

jetbrains-gateway/src/software/aws/toolkits/jetbrains/gateway/CawsConnectorViewPanels.kt

Lines changed: 199 additions & 236 deletions
Large diffs are not rendered by default.

jetbrains-gateway/src/software/aws/toolkits/jetbrains/gateway/CawsLoadingPanel.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,6 @@ abstract class CawsLoadingPanel(protected val lifetime: Lifetime, private val se
146146
lifetime,
147147
CawsSettings().also {
148148
it.initialSpace = CawsSpaceTracker.getInstance().lastSpaceName()
149-
it.cloneType = CawsWizardCloneType.NONE
150149
}
151150
)
152151
)

jetbrains-gateway/src/software/aws/toolkits/jetbrains/gateway/welcomescreen/WorkspaceDataRetriever.kt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ import software.aws.toolkits.jetbrains.gateway.toSourceRepository
1919
import software.aws.toolkits.jetbrains.gateway.toWorkspace
2020
import software.aws.toolkits.jetbrains.services.caws.CawsProject
2121
import software.aws.toolkits.jetbrains.services.caws.listAccessibleProjectsPaginator
22+
import software.aws.toolkits.jetbrains.settings.CawsSpaceTracker
2223
import java.time.Duration
2324
import java.util.concurrent.locks.ReentrantReadWriteLock
2425
import kotlin.concurrent.read
@@ -147,6 +148,9 @@ class WorkspaceDataRetriever(
147148
}
148149

149150
fun markWorkspaceAsDirty(identifer: WorkspaceIdentifier) {
151+
if (CawsSpaceTracker.getInstance().lastSpaceName() != identifer.project.space) {
152+
return
153+
}
150154
dirtyWorkspaces.add(identifer)
151155
updateAlarm.cancelAllRequests()
152156
pollForUpdate()

jetbrains-gateway/src/software/aws/toolkits/jetbrains/gateway/welcomescreen/WorkspaceGroupsPanel.kt

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ import com.intellij.util.ui.components.BorderLayoutPanel
2525
import com.jetbrains.rd.util.lifetime.Lifetime
2626
import software.amazon.awssdk.services.codecatalyst.CodeCatalystClient
2727
import software.aws.toolkits.jetbrains.gateway.CawsSettings
28-
import software.aws.toolkits.jetbrains.gateway.CawsWizardCloneType
2928
import software.aws.toolkits.jetbrains.gateway.SourceRepository
3029
import software.aws.toolkits.jetbrains.gateway.Workspace
3130
import software.aws.toolkits.jetbrains.gateway.cawsWizard
@@ -173,10 +172,6 @@ class WorkspaceGroupsPanel(
173172
CawsSettings().also {
174173
it.project = project
175174
it.linkedRepoName = workspaceGroup.repoName ?: ""
176-
// TODO: 3p unlinked case
177-
if (workspaceGroup.repoName == null) {
178-
it.cloneType = CawsWizardCloneType.NONE
179-
}
180175
}
181176
)
182177
)

resources/resources/software/aws/toolkits/resources/MessagesBundle.properties

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -247,6 +247,7 @@ caws.view.projects_web=View project summary in Amazon CodeCatalyst...
247247
caws.view.workspaces_web=View all Dev Environments for project in Amazon CodeCatalyst...
248248
caws.workspace.backend.title=Amazon CodeCatalyst Dev Environment
249249
caws.workspace.clone.info=The project source repository will be cloned to the Dev Environment directly.
250+
caws.workspace.clone.info_repo=Choose the branch to clone, or create a branch for your work in this Dev Environment.
250251
caws.workspace.clone.ssh_agent=Your SSH agent will be forwarded to the Dev Environment for authentication to the repository.
251252
caws.workspace.connection.failed=Dev Environment connection failed
252253
caws.workspace.creation.failed=Dev Environment creation failed
@@ -258,7 +259,9 @@ caws.workspace.details.branch_new=New Branch:
258259
caws.workspace.details.branch_new_validation=New branch name is required
259260
caws.workspace.details.branch_title=Branch:
260261
caws.workspace.details.branch_validation=Branch selection is required
262+
caws.workspace.details.clone_repo=Clone a repository
261263
caws.workspace.details.create_branch_comment=Choose an existing branch or create a new branch. If you choose a third-party source repository, you must work in an existing branch.
264+
caws.workspace.details.create_empty_dev_env=Create an empty Dev Environment
262265
caws.workspace.details.create_from_existing_branch=Existing Branch
263266
caws.workspace.details.create_from_new_branch=New Branch
264267
caws.workspace.details.developer_tool_settings=Toolkit Developer Settings
@@ -289,6 +292,7 @@ caws.workspace.list_panel_search_empty_text=Search Dev Environments
289292
caws.workspace.new=Create Dev Environment
290293
caws.workspace.panel_other_repos=Other repositories
291294
caws.workspace.settings=Dev Environment Configuration
295+
caws.workspace.settings.repository_header=Repository
292296
caws.write_credentials_to_file_progress=Writing credentials to shared credentials file
293297
cloudformation.capabilities=CloudFormation Capabilities:
294298
cloudformation.capabilities.auto_expand=Auto Expand

0 commit comments

Comments
 (0)