Skip to content

Conversation

@wasade
Copy link
Contributor

@wasade wasade commented Nov 6, 2025

API endpoint to dump comprehensive information about a study. The endpoint is authenticated, and its expected scope of use is within barnacle2 and adjacent systems.

@antgonza antgonza changed the base branch from master to dev November 7, 2025 13:24
Copy link
Member

@antgonza antgonza left a comment

Choose a reason for hiding this comment

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

Really nice @wasade, thank you. Once tests pass I can deploy in qiita-rc.

@wasade
Copy link
Contributor Author

wasade commented Nov 7, 2025

Thanks, Antonio.

I'm not sure how to effectively test locally. There are some reported failures for qiita_pet, but running nosetests locally creates so much output that I can't find the failures in the scroll buffer. Using -q -s or running a second time with --failed doesn't seem to alleviate the output. Is there a strategy that works to avoid all of the debug output and constrain to only the failures?

@coveralls
Copy link

coveralls commented Nov 7, 2025

Coverage Status

coverage: 92.118% (+0.02%) from 92.094%
when pulling ad17f30 on wasade:study_associations
into 1b8b0c7 on qiita-spots:dev.

@wasade
Copy link
Contributor Author

wasade commented Nov 7, 2025

Huh... well, maybe the test failures I saw locally were something else funny

Copy link
Member

@antgonza antgonza left a comment

Choose a reason for hiding this comment

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

I just deployed this in qiita-rc; in case you want to test before we merge.

@antgonza
Copy link
Member

@wasade, is this working as you expected? Is it OK to merge?

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