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

allow fetching of articles that a group has published #377

Open
wants to merge 7 commits into
base: staging
Choose a base branch
from

Conversation

DrumsnChocolate
Copy link
Contributor

@DrumsnChocolate DrumsnChocolate commented Jun 8, 2023

add an attribute to the goup model, allowing the articles of a group to be retrieved via that group's model.

@wilco375 please let me know if the schema looks right to you, I'm not sure why these changes happened (the underlying tables should not be changed by my additions right?)

@DrumsnChocolate
Copy link
Contributor Author

It seems that I need to fix tests :) I will later

@wilco375
Copy link
Contributor

wilco375 commented Jun 8, 2023

I do not see a reason for the schema change, no. But it might just have been caused by changes in a recent dependency update, the changes to it look unrelated to yours.

@DrumsnChocolate
Copy link
Contributor Author

schema changes are explained by #393 (comment)

Copy link

codecov bot commented Oct 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.92%. Comparing base (153319a) to head (b2486ad).

Additional details and impacted files
@@           Coverage Diff            @@
##           staging     #377   +/-   ##
========================================
  Coverage    99.92%   99.92%           
========================================
  Files          203      203           
  Lines         2707     2709    +2     
========================================
+ Hits          2705     2707    +2     
  Misses           2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants