Skip to content

Work in progress for adding internal articles#22

Open
smileservices wants to merge 14 commits intostagingfrom
feature/internal-articles
Open

Work in progress for adding internal articles#22
smileservices wants to merge 14 commits intostagingfrom
feature/internal-articles

Conversation

@smileservices
Copy link
Copy Markdown
Owner

Left to do:

  • update serializers
  • create tests for the API in study_resources/tests/InternalStudyResourceTestCase/py

@Sosshi Sosshi force-pushed the feature/internal-articles branch from fec3742 to 983446a Compare June 23, 2022 21:58
Comment on lines +33 to +35
# todo in this if resource can be created through API not direct
# todo check for the resource content and flag
# todo check if the images have been handled correctly
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Take a look at these todos


def test_api_can_have_discussion(self):
"""test creating a discussion for the internal resource through the api"""
#todo implement this. we want to test the API functionality
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

And this one

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.

3 participants