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 to oscn v60!, refactor using new event dict #53

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jdungan
Copy link
Member

@jdungan jdungan commented Nov 6, 2019

Includes test for api views. Breaks sms app.

@jdungan
Copy link
Member Author

jdungan commented Nov 6, 2019

Move my new tests to a directory. Once merged, we can move the tests.py into the directory.

@jdungan jdungan requested a review from syk1112 November 6, 2019 19:12
@jdungan jdungan changed the title update to oscn v59, refactor using new event dict update to oscn v60!, refactor using new event dict Nov 10, 2019
@kendallcorner
Copy link
Collaborator

I merged with master (to run the tests from those newer commits), and they run fine, so it LGTM, but I'll let someone else push the button 😬

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