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

Describe how to use Localstack to improve the development experience with AWS #6875

Merged
merged 13 commits into from
Nov 6, 2024

Conversation

saratry
Copy link
Contributor

@saratry saratry commented Oct 29, 2024

No description provided.

Comment on lines 2 to 3
title: AWS local development
summary: Guidance on local development when using AWS
Copy link
Member

Choose a reason for hiding this comment

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

Although this won't be in the nav, so it would be hard to get here without manually altering the URL, I would imagine that any other article added to this section would not be about local development but more AWS stuff in general. Shouldn't this title/summary be more general as well?

Copy link
Member

Choose a reason for hiding this comment

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

I don't like the need of this index page. I see two options:

  1. Delete this page and the aws folder. Move the nservicebus/aws/local-development.md under architecture/aws/local-development.md and keep the menu links as is pointing to the new location
  2. Delete this page and the aws folder. Move the nservicebus/aws/local-development.md under transports/sqs/local-development.md and keep the menu links as is pointing to the new location

Copy link
Member

@DavidBoike DavidBoike Nov 4, 2024

Choose a reason for hiding this comment

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

I prefer option 2. It's a greater affinity for SQS. If you're using Dynamo it should be because you're already using Dynamo and you're really past the point of developing a plan for these things.

Copy link
Member

Choose a reason for hiding this comment

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

@DavidBoike@saratry and I thought the index page could look like a landing/summary page for supported AWS services. What do you think?

nservicebus/aws/local-development.md Outdated Show resolved Hide resolved
nservicebus/aws/local-development.md Outdated Show resolved Hide resolved
nservicebus/aws/local-development.md Outdated Show resolved Hide resolved
servicecontrol/transports.md Show resolved Hide resolved
nservicebus/aws/local-development.md Outdated Show resolved Hide resolved
nservicebus/aws/local-development.md Outdated Show resolved Hide resolved
nservicebus/aws/local-development.md Outdated Show resolved Hide resolved
Comment on lines 2 to 3
title: AWS local development
summary: Guidance on local development when using AWS
Copy link
Member

Choose a reason for hiding this comment

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

I don't like the need of this index page. I see two options:

  1. Delete this page and the aws folder. Move the nservicebus/aws/local-development.md under architecture/aws/local-development.md and keep the menu links as is pointing to the new location
  2. Delete this page and the aws folder. Move the nservicebus/aws/local-development.md under transports/sqs/local-development.md and keep the menu links as is pointing to the new location

nservicebus/aws/local-development.md Show resolved Hide resolved
nservicebus/aws/index.md Outdated Show resolved Hide resolved
nservicebus/aws/index.md Outdated Show resolved Hide resolved
Comment on lines 2 to 3
title: AWS local development
summary: Guidance on local development when using AWS
Copy link
Member

Choose a reason for hiding this comment

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

@DavidBoike@saratry and I thought the index page could look like a landing/summary page for supported AWS services. What do you think?

@mauroservienti mauroservienti force-pushed the localstack branch 2 times, most recently from 09b9ee3 to e1249ec Compare November 5, 2024 14:52
@mauroservienti mauroservienti marked this pull request as ready for review November 5, 2024 15:08
@DavidBoike DavidBoike merged commit 9d2d586 into master Nov 6, 2024
4 checks passed
@DavidBoike DavidBoike deleted the localstack branch November 6, 2024 15:42
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.

3 participants