search

krux / starport

Apache License 2.0
2 stars 7 forks source link

Fix issue 31 #33

Closed gcagrici closed 5 years ago

gcagrici commented 5 years ago

Make SNS ARN default for owner notification under krux.starport.notification.sns_owner

talbright commented 5 years ago

@sushengloong this should fix the issue you ran into last night and I hit this AM.

realstraw commented 5 years ago

@gcagrici let's make sure to have more useful commit title during merge, and do some clean up for the commit message, these will be automatically put into the releases, they would look confusing.

sushengloong commented 5 years ago

@talbright - yes, bin/submit.sh is working for me now! Can you try again on your end?

@gcagrici - thanks!

talbright commented 5 years ago

@sushengloong I verified it on my side as well, looks good.