search

csvalpha / amber-api

https://csvalpha.nl
MIT License
4 stars 4 forks source link

allow fetching of articles that a group has published #377

Open DrumsnChocolate opened 1 year ago

DrumsnChocolate commented 1 year ago

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 commented 1 year ago

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

wilco375 commented 1 year ago

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 commented 1 year ago

schema changes are explained by https://github.com/csvalpha/amber-api/pull/393#discussion_r1382373606

codecov[bot] commented 4 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

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

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.