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

Update postgres plan in Heroku deployment docs #5383

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

theptrk
Copy link
Contributor

@theptrk theptrk commented Sep 17, 2024

Description

Changes the command to create a heroku postgres addon

$ heroku addons:create heroku-postgresql:mini

 ›   Warning: heroku update available from 8.11.5 to 9.2.1.
Creating heroku-postgresql:mini on ⬢ location-tracker... !
 ▸    We tried to create heroku-postgresql:mini, but received an error from the add-on provider. Try the request again, or log a support ticket and include
 ▸    this message: The mini plan has reached end-of-life. You can try again with a valid plan. See the available plans with: heroku addons:plans
 ▸    heroku-postgresql.

The next cheapest heroku option is "essentials-0"

documentation: https://elements.heroku.com/addons/heroku-postgresql

$ heroku addons:create heroku-postgresql:essential-0

 ›   Warning: heroku update available from 8.11.5 to 9.2.1.
Creating heroku-postgresql:essential-0 on ⬢ location-tracker... ~$0.007/hour (max $5/month)
Database should be available soon
postgresql-silhouetted-14783 is being created in the background. The app will restart when complete...
Use heroku addons:info postgresql-silhouetted-14783 to check creation progress
Use heroku addons:docs heroku-postgresql to view documentation

Checklist:

  • I've made sure that tests are updated accordingly (especially if adding or updating a template option)
  • I've updated the documentation or confirm that my change doesn't require any updates

Rationale

This option (mini) is now longer valid

This option is now longer valid

```
$ heroku addons:create heroku-postgresql:mini

 ›   Warning: heroku update available from 8.11.5 to 9.2.1.
Creating heroku-postgresql:mini on ⬢ location-tracker... !
 ▸    We tried to create heroku-postgresql:mini, but received an error from the add-on provider. Try the request again, or log a support ticket and include
 ▸    this message: The mini plan has reached end-of-life. You can try again with a valid plan. See the available plans with: heroku addons:plans
 ▸    heroku-postgresql.
```

The next cheapest heroku option is "essentials-0"

```
$ heroku addons:create heroku-postgresql:essential-0

 ›   Warning: heroku update available from 8.11.5 to 9.2.1.
Creating heroku-postgresql:essential-0 on ⬢ location-tracker... ~$0.007/hour (max $5/month)
Database should be available soon
postgresql-silhouetted-14783 is being created in the background. The app will restart when complete...
Use heroku addons:info postgresql-silhouetted-14783 to check creation progress
Use heroku addons:docs heroku-postgresql to view documentation
```

documentation: https://elements.heroku.com/addons/heroku-postgresql
@browniebroke browniebroke changed the title Update deployment-on-heroku.rst Update postgres plan in Heroku deployment docs Sep 17, 2024
@browniebroke browniebroke merged commit caae3ce into cookiecutter:master Sep 17, 2024
14 checks passed
@browniebroke
Copy link
Member

Thanks! Notes that a while back but completely forgot it was in our docs.

@theptrk
Copy link
Contributor Author

theptrk commented Sep 30, 2024

Might have to revert this. We gain the nav bar but the way messages are set up, now you see two of them.

Messages from allauth entrance.html and messages from base.html

Screenshot 2024-09-29 at 7 18 27 PM Screenshot 2024-09-29 at 7 19 45 PM

One solution may be to move the messages into main, but I'm not sure if that serves the original purpose of main.html

    {% block main %}
      {% if messages %}
        {% for message in messages %}
          <div class="alert alert-dismissible {% if message.tags %}alert-{{ message.tags }}{% endif %}">
            {{ message }}
            <button type="button"
                    class="btn-close"
                    data-bs-dismiss="alert"
                    aria-label="Close"></button>
          </div>
        {% endfor %}
      {% endif %}
      {% block content %}
        <p>Use this document as a way to quick start any new project.</p>
      {% endblock content %}
    {% endblock main %}

@theptrk
Copy link
Contributor Author

theptrk commented Sep 30, 2024

This would be an updated version that moves messages into main

#5410

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants