Bradley Wagner bradical

Organizations

@hannonhill
@bradical

Thanks for filing. I'll check but I think remember compiling this plugin for Apache 2.4 in our cloud environment. Let me check on that and get back…

@bradical

Looks like they'll be patching it in a future desktop version: https://community.spotify.com/t5/Live-Ideas/Bring-back-Applescript-support/idc-p/106…

@bradical

Quit both Alfred and Spotify. Apply the fix. Restart both programs. It didn't work for me until I restarted Alfred.

@bradical

Nice. Restarting Alfred did the trick. I restarted Spotify multiple times to no avail but didn't realize Alfred restart was necessary. Thanks!

@bradical

@matbrady did this work for you in the most recent release 1.0.2.6.g9977a14b? It worked for me in the previous version, but didn't work with this one.

@bradical

Doh, I think this broke again in 1.0.2.6.g9977a14b and applying the change to the Info.plist didn't work this time. Anyone else have any luck?

@bradical

Yep, that did the trick for me. Thanks!! On Fri, Mar 13, 2015 at 11:27 AM, John Gilbert notifications@github.com wrote: It's glitch in .plist, it'…

@bradical

Quick update for Fri, 3/13/2015: We're working on a fix to email rendering hannonhill/Spectate#2306 that will address the weird character issues w…

@bradical
Instantiating a new SpectateTrigger from within SpectateTrigger#invoke makes for weird debugging
@bradical

Closing this out.

@bradical
Make trigger skip jobs quickly
@bradical

Closing this out.

@bradical

Closing in favor of hannonhill/Spectate#2306. Note, that this could pop up for production emails with extremely long line-lengths.

@bradical
Email render issues
@bradical
  • @bradical d70c918
    Have to include WS stubs in compiled JAR for it to work #7
  • @bradical a3cc6f3
    added back WS code generation to build file and classpath. Ensured ja…
  • 30 more commits »
@bradical

This is fine but FYI that Exception encompasses IOException

@bradical

This is more convention but I rarely use this when making method calls as this is assumed. I'll use it when calling super

@bradical
  • @bradical 283c2cf
    Removed unread private variables and unused imports #2
@bradical

Probably don't want to eat this error and continue here. I would just remove this try/catch and make the method throw Exception so the errors bubbl…

@bradical

Why the repeated method call here as opposed to just using the local variable?

@bradical

You're putting stuff into this map but it doesn't look like your'e ever pulling stuff out of it. Seems like if you got rid of this campaigns map yo…

@bradical

I know these changes didn't introduce this code but this issue seemed like a good place to address it. See comments here and here

@bradical

You definitely don't want to be parsing result HTML. If there's something you're trying to read, use a JSON endpoint to get it. There's no guarante…

@bradical

@nadirayasmeen I'm not really following this code block here. In particular, this call getLeads seems odd.

@bradical

The comment has since been removed.

@bradical

@nadirayasmeen I don't think you committed your new file.

@bradical

Quick update here. We recently deployed a build of 3d9d14f to their dev server and have been testing that. We'll need to continue to test what we'…

@bradical

Nadira is actively working on this one.