Nick Sweeting

Results 10 comments of Nick Sweeting

Just thought I would post another alternative. With my projects, I have been using my [GenQueue](https://github.com/nsweeting/gen_queue) abstraction to work with different forms of background job queues. Typically, we might have...

Sounds good @IanLuites. Thanks for the update. Perhaps we can sort out any behaviour conflicts that might occur from the amqp update, then create a new PR to run the...

From what I understand, this PR just matches existing behaviour. The older version of AMQP would simply raise a MatchError itself. Whereas here - it will be raised within TaskBunny....

I would maybe try and keep the PR focused on a simple hmac authentication method. One that could be applied against both oauth and webhooks. Something akin to `Shopify.HMAC.authenticate(secret, raw_body)`...

At the moment - the the nested module for sure seems like the cleaner approach. Personally, I haven't yet had a need to fetch those nested resources yet. I would...

To add on to the this further. I'm basically trying to envision a few different ways we could handle fetching nested relations through pipelines. Although it creates "extra" structs in...

Regarding adding `:headers` to the `Shopify.Response` struct - that would be great. Exactly how I envisioned fixing this issue - just havent had time. A PR would be much appreciated...

@Odaeus Thanks for the suggestion. I'm wondering if you could provide a bit more context on the use case you have where an outside process is needing to interact with...

Yeah - I think adding `sync_start` would be the best solution for now as it mirrors other parts of the application as well. Absolutely open to PR's! Thanks!

@Odaeus This looks great! In terms of testing - I'm wondering if we should add an extra attribute to the internal server state - something akin to `:sync_start_run` that defaults...