Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Serverless offline 6 #125

Closed
wants to merge 13 commits into from
Closed

Conversation

esetnik
Copy link
Contributor

@esetnik esetnik commented Apr 28, 2020

Resolves #116, #122

Note that to achieve this there's a pretty big hack. I need to access the private class property #lambda. See https://github.com/CoorpAcademy/serverless-plugins/pull/125/files#diff-32e9d7d10726a2f50f1aea57d090602cR196

@esetnik
Copy link
Contributor Author

esetnik commented May 4, 2020

Note for maintainers. I can’t get the python test to pass. Can someone who knows about running python on serverless take a look?

@@ -56,6 +53,14 @@ class ServerlessOfflineSQS {
'before:offline:start:end': this.offlineStartEnd.bind(this)
};

this.serverlessOfflinePlugin = this.serverless.pluginManager.plugins.find(

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For me, this doesn't find anything even though I do have serverless-offline-sqs plugin installed... Are you sure this is the best way to find it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I couldn't find any better way to hook into the serverless-offline plugin. The plugin order matters. Make sure that - serverless-offline appears before - serverless-offline-sqs in your serverless.yml

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup, that was it - after changing the order I can approve that it works on my side. (tested with serverless-offline versions 6.1.0 and 6.1.4)

@esetnik
Copy link
Contributor Author

esetnik commented May 11, 2020

@AdrieanKhisbe can you take a look please?

@ghost
Copy link

ghost commented May 19, 2020

Any plans to look at this soon?

@johanpedroo
Copy link

waiting for this :(

@YOU54F
Copy link

YOU54F commented Jun 13, 2020

Thanks for this @esetnik, this is working for me against serverless-offline v6.3.0 running against a node 12 service running locally

I published a version on npm as npm i serverless-offline-sqs-elastic whilst we wait for this to get merged into a main branch.

https://www.npmjs.com/package/serverless-offline-sqs-elastic

@christophersbarrett
Copy link

christophersbarrett commented Jun 15, 2020

Never mind my previous response, I get it now. It does seem to alleviate the main error but I haven't had a chance to actually test the functionality yet :-D

@godu
Copy link
Contributor

godu commented Jun 16, 2020

Hi, i'm working on full rewrite to support serverless v6.
Take a look of #129

@godu godu closed this Jun 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

serverless-offline-sqs incompatible with new versions of serverless-offline
6 participants