Create a tweet when submitting a CFP asynchronous about cfpexchange HOT 9 CLOSED

Thanks @Jandev, wasn't sure of the current state of the ARM templates. Will add them there.

from cfpexchange.

Boom!

from cfpexchange.

I'll pick this one up.

from cfpexchange.

Hi, quick question! I'm working on this issue, and I encountered the DownloadEventImageMessageSender class that sends the message for downloading event images.

It contains this code:

var queueClient = new QueueClient(
    new ServiceBusConnectionStringBuilder(servicebusConnectionstring),
    ReceiveMode.ReceiveAndDelete);

[...]

await queueClient.SendAsync(message);

I'm curious why the ReceiveMode is explicitly set, while it's an optional parameter and we're just sending messages here.

Long story short: should it stay or should it go?

EDIT:
I think it's done. I didn't specify the ReceiveMode for sending the Twitter service bus message. There's a setting that needs to be added to the website, and several (the Twitter ones) to the Function App. Can I create a PR so you can take care of that part? It's easier with access to the Azure subscription 🤓

from cfpexchange.

I think the ReceiveMode isn't necessary indeed, it doesn't make sense over here.

There's a setting that needs to be added to the website and several (the Twitter ones) to the Function App

A setting you can't set in ARM?
In the past, I've pinged @jfversluis in my PR with a step-by-step plan to set settings like that.

from cfpexchange.

Can I create a PR so you can take care of that part? It's easier with access to the Azure subscription 🤓

Can I add people to just a part already in Azure? In that case I'll be happy to set something up if that makes your life easier ;)

What else do you need from me at this point?

from cfpexchange.

Can I add people to just a part already in Azure?

Yes, you can! I'll try to add the settings to the ARM template though. Shouldn't be an issue.

from cfpexchange.

One setting added, moved some around. Should be OK.
Side note: I now reference a secret in Key Vault (as is the case with a similar setting) which isn't in there and doesn't look like it's added through ARM. So please take that into account 😄.
The PR is live: #109 .

from cfpexchange.

Since PR #109 has been merged, it looks like this issue can be closed. 🤓

from cfpexchange.