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

C4 instance type support #638

Closed
wants to merge 5 commits into
base: trunk
from

Conversation

Projects
None yet
3 participants
@amitlalani
Contributor

amitlalani commented Nov 22, 2015

No description provided.

@tonybaloney

This comment has been minimized.

Show comment
Hide comment
@tonybaloney

tonybaloney Nov 22, 2015

Contributor

looks good, will merge

Contributor

tonybaloney commented Nov 22, 2015

looks good, will merge

@tonybaloney

This comment has been minimized.

Show comment
Hide comment
@tonybaloney

tonybaloney Nov 22, 2015

Contributor

@amitofs any further commits?

Contributor

tonybaloney commented Nov 22, 2015

@amitofs any further commits?

@tonybaloney

This comment has been minimized.

Show comment
Hide comment
@tonybaloney

tonybaloney Nov 22, 2015

Contributor

Pricing also needs to be updated as per #634

Contributor

tonybaloney commented Nov 22, 2015

Pricing also needs to be updated as per #634

@tonybaloney

This comment has been minimized.

Show comment
Hide comment
@tonybaloney

tonybaloney Nov 22, 2015

Contributor

nw, @Kami already did it on the last sync, the prices are already there.

Contributor

tonybaloney commented Nov 22, 2015

nw, @Kami already did it on the last sync, the prices are already there.

@asfgit asfgit closed this in ee4f7ef Nov 22, 2015

@Kami

This comment has been minimized.

Show comment
Hide comment
@Kami

Kami Nov 22, 2015

Member

@tonybaloney Just FYI, looks like this (29e3387) slipped through since the patch was merged before @amitofs updated the PR.

In any case, it's fixed now :)

Member

Kami commented Nov 22, 2015

@tonybaloney Just FYI, looks like this (29e3387) slipped through since the patch was merged before @amitofs updated the PR.

In any case, it's fixed now :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment