-
Notifications
You must be signed in to change notification settings - Fork 24
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 ability to query for active articles via NewsArticle model #731
Conversation
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.
Nice job on this, this LGTM!
FYI, @schroerbrian, @lgarofalo, @jjfreund, the next time we deploy, we'll need to deploy both this and the associated PR, ShelterTechSF/askdarcel-web#1319, at the same time, since this one changes the default behavior of the /news_articles
index endpoint to return all articles, not just the active one. I think this is the right change to make, as opposed to trying to do it in a backwards-compatible way, but we just have to deploy both at the same time.
BTW, you're failing the lint checks, so you should fix them up. It looks like the failures are autofixable, so if you just run |
…eating a new user
…and create user endpoints to user controller
…ser. Ensure auth0 userId returned from validation hit matches user id sent from front-end
…me fairly complicated integration with auth0 environment
These changes are needed for this Breaking News Edit Page UI enhancements.