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

Add AWS resources #381

Merged
merged 2 commits into from
Feb 10, 2021
Merged

Add AWS resources #381

merged 2 commits into from
Feb 10, 2021

Conversation

semnil
Copy link
Contributor

@semnil semnil commented Nov 28, 2020

I'm using the results of this wonderful project a lot. Thank you. 😄

Add AWS resources (from the AWS architecture icons).
I may have added too many at once. 😅
If the file names aren't right, I'll fix it!

Closes #272

Comment on lines 439 to 441
- **diagrams.aws.network.ElbApplicationLoadBalancer**
- **diagrams.aws.network.ElbClassicLoadBalancer**
- **diagrams.aws.network.ElbNetworkLoadBalancer**
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Perhaps removing the "Elb" prefix might be more user-friendly. 🤔

Copy link
Owner

Choose a reason for hiding this comment

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

How about alias these nodes?

@semnil semnil marked this pull request as ready for review November 28, 2020 10:31
@wolfspyre
Copy link
Contributor

This adds a lot of value.. @mingrammer - Thoughts?

Copy link
Owner

@mingrammer mingrammer left a comment

Choose a reason for hiding this comment

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

Looks awesome! I have some suggestions.

It would be good to add aliases for some nodes.

AMI for EC2Ami
ALB for ElbApplicationLoadBalancer
ELB for ElbClassLoadBalancer
NLB for ElbNetworkLoadBalancer

@mingrammer mingrammer added area/provider/aws AWS provider comp/node Issue of node component status/accepted Accepted proposal labels Jan 28, 2021
@mingrammer
Copy link
Owner

@semnil @wolfspyre Sorry for the late review :( I left a review comment.

@semnil semnil force-pushed the add-aws-resources branch from 0216e51 to d054783 Compare January 30, 2021 06:30
@semnil
Copy link
Contributor Author

semnil commented Jan 30, 2021

@mingrammer Thank you for your review!
I added the suggested aliases. 😄

@semnil semnil requested a review from mingrammer January 30, 2021 06:32
Copy link
Owner

@mingrammer mingrammer left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you.

@mingrammer mingrammer merged commit 8abbf8a into mingrammer:master Feb 10, 2021
@semnil semnil deleted the add-aws-resources branch February 10, 2021 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/provider/aws AWS provider comp/node Issue of node component status/accepted Accepted proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing AWS smaller resources in storage
3 participants