Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Regression: Open in Portal and copying to clipboard do not work on Linux #699

Closed
v-dantch opened this issue Dec 11, 2018 · 5 comments
Closed

Regression: Open in Portal and copying to clipboard do not work on Linux #699

v-dantch opened this issue Dec 11, 2018 · 5 comments
Assignees
@StephenWeatherford
Labels
AT-CTI P1
Milestone
0.6.0

Comments

@v-dantch
Copy link

OS: Linux
Node.js: V10.14.1
Extension Build: 20181209.1

Repro Steps:

  1. Drill down from 'Registries' -> 'Azure' -> Right click one registry and select 'Browse in the Azure Portal'.
  2. Check the result.

Expect:
Succeed to browse the Azure registry in portal.

Actual:
Fail to browse the azure registry in portal.
@StephenWeatherford
Copy link
Contributor

Works fine on windows, does nothing on Linux.

@StephenWeatherford StephenWeatherford added this to the 0.7.0 milestone Jan 16, 2019
@StephenWeatherford
Copy link
Contributor

Eric might know what's required.

@StephenWeatherford StephenWeatherford added P1 and removed P2 labels Feb 7, 2019
@StephenWeatherford StephenWeatherford self-assigned this Feb 7, 2019
@StephenWeatherford StephenWeatherford changed the title Fail to browse the azure registry in portal on Linux Regression: Fail to browse the azure registry in portal on Linux Feb 7, 2019
@StephenWeatherford StephenWeatherford changed the title Regression: Fail to browse the azure registry in portal on Linux Regression: Open in Portal and copying to clipboard do not work on Linux Feb 7, 2019
@StephenWeatherford
Copy link
Contributor

Two issues:

  1. opn needs to be excluded from webpack
  2. fallback exe in node_modules/opn/ doesn't have execute bit set ([Feature] keeping original whole or partial stats webpack-contrib/copy-webpack-plugin#35)

@PrashanthCorp
Copy link
Contributor

@v-dantch can you verify if the bug still repros?

@v-dantch
Copy link
Author

v-dantch commented Feb 27, 2019 •
edited
Loading

Hi @PrashanthCorp we verified this issue on Linux with the latest Docker Extension build 20190226.1. And this issue is fixed.

@vscodebot vscodebot bot locked and limited conversation to collaborators Apr 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@StephenWeatherford StephenWeatherford

Labels
AT-CTI P1
Projects
None yet
Milestone
0.6.0
Development

No branches or pull requests
3 participants
@StephenWeatherford @PrashanthCorp @v-dantch