-
Notifications
You must be signed in to change notification settings - Fork 27
Update sample apps readme #196
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
Conversation
| Using AWS CLI: | ||
| ``` | ||
| $messageBody = "{""""type"""":""""chatMessage"""",""""id"""":""""123"""",""""source"""":""""test"""",""""specversion"""":""""1.0"""",""""time"""":""""2024-01-01T00:00:00Z"""",""""data"""":""""{\\""""messageDescription\\"""":\\""""Test message\\""""}""""}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i was trying for an hour or so to figure out how to make this command better to create the messagebody but was having issues with the serialization. creating the string manually was the easiest way for now. will have to come back to this
| <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="6.0.*" /> | ||
| <PackageReference Include="Microsoft.Extensions.Hosting" Version="6.0.*" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: use version 8 of these packages
normj
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved but throwing out the idea that instead of walking users through how to setup the Lambda test tool we could create an Aspire AppHost project and use our Aspire integration for helping users get started. Especially after merge in the work to Aspire for SQS event source support.
i will make a backlog item for this. i like the idea too |
Issue #, if available: DOTNET-7436
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.