[Bug]: Filter news by site partners #456
Labels
bug
Something isn't working
good first issue
Should not need in depth knowledge of code base to complete
v
Valuable
Milestone
Comments
katjam
added
the
good first issue
|
I think its OK to just show everything for now? Obviously nice to have the filter option but it doesn't feel high priority to me. |
|
Agree. It will only be a problem if people start using it. I think the name resolution from the Partners authoring the story won't work if the partners are not included in partnership tags for Trans Dimension but probably will not be noticeable because "I think" they will show as an ID number. We might want to check this though, in case it breaks more dramatically than that.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Currently we do include the partner in News data but I don't think we filter out News Items that belong to partners outside the partnership.
This can be handled in
src/Data/Placecal/Articles.elmbecause we have access to the partner list here.The text was updated successfully, but these errors were encountered: