-
Notifications
You must be signed in to change notification settings - Fork 132
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
Set peoples status through api #294
Conversation
- This introduces a more flexible way of storing the "now doing" activities - Also extracts helpers into utils module - And adds tests
@pwalsh I think this is ready for a first review now. |
I'm deploying this to staging already to see if it works. |
Changes Unknown when pulling 4f82047 on 292-set-staff-member-statuses into * on master*. |
The last four commits clean up the code some more and add functional tests for the new |
If we merge #305 before this we need to change the migration to become number |
Assigning @pwalsh as this awaits review now |
This PR will collect the changes that are needed for making us more relatable.
It is work in progress right now but will fix #292
This goes hand in hand with the changes in okfns nice chatabot that is developed on the make-mowglibot-relatable branch.