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

implemented "show trace view" feature to open the trace view in browser #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

timwillsie
Copy link

Hey there.

Looking for an alternative for the apigee online editor I stumbled upon your sublime package. Great work! Exactly what I was looking for.

I thought it would be useful to have the trace tool just one click away since this would be the first thing I would open once I deploy a new proxy...
So I added a new command which opens a configured uri (configurable in deploy_vars.json, automatically customized with the organization and proxy name) in a new tab of the system standard browser.

I'm a total python noob, so my code might not be as clean as it could be. I appreciate any advice!

I successfully tested the feature in OSX and Windows 7.

What do you think about it?

Cheers, Tim.

@girishbr
Copy link
Contributor

Hey Tim,
Thanks for trying out SweetLime and sending in a pull request.
Will review and merge this by mid-week.
Regards,Girish

@@ -24,7 +24,7 @@ OSX
> `git clone [email protected]:apigee/Sweetlime.git ~/Library/Application\ Support/Sublime\ Text\ 3/Packages/ApigeeSweetLime'

Windows
> coming soon (testing underway)
> `git clone [email protected]:apigee/Sweetlime.git %APPDATA%\Sublime Text 3\Packages\ApigeeSweetLime'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works in Windows too? thats great!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well, at least with windows 7 and the latest sublime text 3 version. Unfortunately I have no chance to test this on windows 8. But I think it's a good idea to give the installation advice in the readme, so anyone with windows 8 has any clue how to test it and open an issue if it doesn't work.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants