Skip to content
This repository has been archived by the owner on Nov 28, 2022. It is now read-only.

SVT: Appsody Python Flask projects are being imported with Java as the language #486

Closed
sujeilyfonseca opened this issue Sep 18, 2019 · 6 comments
Assignees
Labels
area/vscode-ide Issue associated with the Codewind VS Code/Theia extension kind/bug priority/stopship

Comments

@sujeilyfonseca
Copy link

sujeilyfonseca commented Sep 18, 2019

Codewind version: 0.4.0
OS: Windows 10 and MacOS Mojave (10.14.6)

IDE extension version: 0.4.0
IDE version: 1.38.1

Description: Due to eclipse/codewind#445, appsodyExtension must be selected manually to properly import a project into VS Code.

However, Appsody Python Flask projects are being imported with Java as the language, after selecting appsodyExtension.

Screen Shot 2019-09-18 at 5 23 18 PM

Screen Shot 2019-09-18 at 5 24 18 PM

Screen Shot 2019-09-18 at 5 24 27 PM

Steps:

  1. Install Codewind
  2. Start Codewind
  3. Create an Appsody Python Flask project
  4. Remove the created project from Codewind
  5. Import the project
  6. Select appsodyExtension
@sujeilyfonseca sujeilyfonseca changed the title SVT: Appsody Python Flask projects are being imported with Java as a language SVT: Appsody Python Flask projects are being imported with Java as the language Sep 18, 2019
@tetchel
Copy link

tetchel commented Sep 18, 2019

yep, i see the bug. there is a logic error in this 'wizard' for extension project types

@tetchel tetchel self-assigned this Sep 18, 2019
@tetchel tetchel added the area/vscode-ide Issue associated with the Codewind VS Code/Theia extension label Sep 18, 2019
tetchel pushed a commit to tetchel/codewind-vscode that referenced this issue Sep 18, 2019
@mmulholl
Copy link

mmulholl commented Sep 18, 2019

Seeing this issue with all appsody project types using eclipse.

For example when importing an existing Appsody Node Skaffold project, by default the type is set to Node.js - if i install that way it will not build. However, If I change the type to appsody and language to Node.js all is fine.

Making the stop/ship because it is both bad and a regression.

@tetchel
Copy link

tetchel commented Sep 18, 2019

please open a different bug for eclipse, there was an error in the vs code logic which is what this issue is for (and it's fixed now)

@mmulholl
Copy link

will do

@makandre
Copy link
Contributor

makandre commented Sep 18, 2019

Since we are talking about python, this is possibly related to #398 as well

Is the eclipse bug this? eclipse-archived/codewind-eclipse#307

jopit pushed a commit to eclipse-archived/codewind-vscode that referenced this issue Sep 19, 2019
tetchel pushed a commit to eclipse-archived/codewind-vscode that referenced this issue Sep 19, 2019
@sujeilyfonseca
Copy link
Author

I tested this issue on 2019-09-19 and I no longer see the problem.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/vscode-ide Issue associated with the Codewind VS Code/Theia extension kind/bug priority/stopship
Projects
None yet
Development

No branches or pull requests

4 participants