-
Notifications
You must be signed in to change notification settings - Fork 9.5k
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
Fixing heroku read for organization app #668
Fixing heroku read for organization app #668
Conversation
7da3076
to
bf410c0
Compare
I removed some unnecessary logic + added mandatory fields to be set about organization (locked, # and private) |
bf410c0
to
9b022b7
Compare
… have it in a private area instead
9b022b7
to
2aa3110
Compare
Hi there - thanks for this! Can you lay out for me some steps to take to smoke test this? Then I can verify it and pull it in if all looks good. 👍 |
d.Set("git_url", app.App.GitURL) | ||
d.Set("web_url", app.App.WebURL) | ||
d.Set("config_vars", configVarsValue) | ||
d.Set("all_config_vars", app.Vars) | ||
if organizationApp { | ||
d.Set("organization.#", "1") |
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.
this line could be dropped with recent fix for the map (count no longer stored)
the situation might occur eg:
|
Working with Heroku Support to set up an organization for Terraform acceptance testing. Will follow up when I've heard back from them. |
@phinze you heard back from heroku? anything we can do to move this forward? |
I had a long and meandering support ticket that resulted in a test organization being allocated, but now logging into my account today it's gone again. 😞 Re-upped with a fresh support ticket and we'll see what happens. If anybody from the community w/ an account can test this and report 👍 I'm happy to pull it in on your word. Otherwise I'll check back in when I get a fresh org acct. |
Re-submitted this as #4495 - finally landing it! 😅 |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
it will help in managing state when transitioning from org app to private and back.