Skip to content

current_user subscribed? too specific#4

Open
ChrisCPO wants to merge 1 commit intomasterfrom
chris_ambiguity
Open

current_user subscribed? too specific#4
ChrisCPO wants to merge 1 commit intomasterfrom
chris_ambiguity

Conversation

@ChrisCPO
Copy link
Copy Markdown
Contributor

@ChrisCPO ChrisCPO commented Nov 1, 2016

  • subscribed? => paywall_access?

* subscribed? => paywall_access?
Copy link
Copy Markdown
Member

@kevinjasker kevinjasker left a comment

Choose a reason for hiding this comment

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

I don't know if we have this pinned in the gemfile, but merging this before the associated change in webapp will break deploys.

@ChrisCPO
Copy link
Copy Markdown
Contributor Author

ChrisCPO commented Nov 1, 2016

ah, wait how often does heroku run bundle install?

@kevinjasker
Copy link
Copy Markdown
Member

It gets run on every deploy, afaik

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.

2 participants