bullexjacob212 / walkaround

Automatically exported from code.google.com/p/walkaround
Apache License 2.0
0 stars 0 forks source link

Suggestion - add "Import by wave id" feature. #28

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Sometimes you just want to import a wave fast by wave id. It would be great to 
have such an option IMO.

Original issue reported on code.google.com by vega113 on 15 Dec 2011 at 7:56

GoogleCodeExporter commented 8 years ago
Good idea.  It should be possible to add another form to 
ImportOverviewFragment.gxp, something like

                  <form method='post' action='import'>
                    <input type='hidden' name='action' value='importwave'/>
                    <input name='instance' value='XXX'/>
                    <input name='waveid' value='YYY'/>
                    <input type='hidden' name='token' expr:value='token'/>
                    <input type='submit' value='import'/>
                  </form>

I don't think ImportOverviewHandler even needs to change.  Feel free to play 
around with this and send a patch.

Original comment by oh...@google.com on 15 Dec 2011 at 8:05

GoogleCodeExporter commented 8 years ago
Yeah, I can try. But is there an easy way to display the /import page while 
running the app in local dev mode wihtout being asked to enable pictures?

Original comment by vega113 on 15 Dec 2011 at 12:55

GoogleCodeExporter commented 8 years ago
I think you do need to click "enable" and go through the OAuth authorization, 
since import depends on OAuth to access your waves.  If the OAuth pop-up gives 
you errors, go to the Google API console and make sure 
http://localhost:8080/authenticate is in the list of authorized redirect URIs 
as described in http://code.google.com/p/walkaround/wiki/RunningTheCode .

Original comment by oh...@google.com on 15 Dec 2011 at 6:32

GoogleCodeExporter commented 8 years ago
I ended up with something like this:
https://code.google.com/r/vega113-wavewa/source/detail?r=4a7f2617be71c95e51ed50d
1cc82623bd42356d8&name=waveidimport
Probably, could add some javascript in order to avoid duplicating the instance 
selection code, but wasn't sure it worth the effort.

Original comment by vega113 on 17 Dec 2011 at 8:35

GoogleCodeExporter commented 8 years ago
When I tried
git clone https://code.google.com/r/vega113-wavewa/
I did not get that change.  Which repository is it in?  Or is it not committed?

Original comment by oh...@google.com on 19 Dec 2011 at 8:40

GoogleCodeExporter commented 8 years ago
It's in waveidimport branch.

Original comment by vega113 on 19 Dec 2011 at 9:10

GoogleCodeExporter commented 8 years ago
Found it, thanks.  I pulled from your branch, which means your tweaks to 
.project and .classpath are now also integrated.

Thanks for implementing this feature!

Original comment by oh...@google.com on 19 Dec 2011 at 10:37