Skip to content
This repository has been archived by the owner on Nov 7, 2019. It is now read-only.

apps: Add newsletters app #720

Merged
merged 3 commits into from
May 27, 2019
Merged

apps: Add newsletters app #720

merged 3 commits into from
May 27, 2019

Conversation

rmader
Copy link
Contributor

@rmader rmader commented May 23, 2019 •
edited
Loading

Not sure if this is complete yet (no tests yet), but probably this can get merged as is.

@rmader rmader force-pushed the rm-2019-05-newsletter branch 2 times, most recently from 1a97346 to 286c0c7 Compare May 23, 2019 15:09
@rmader rmader changed the title WIP: newsletter app apps: newsletter app May 23, 2019
@rmader rmader changed the title apps: newsletter app apps: Add newsletters app May 23, 2019
@fuzzylogic2000
Copy link
Contributor

I think, it works now, but only for project follows and we wanted project participants!
How do we get them all? I wanted to start with the comments, but got a bit lost on how to find the projects there (filter(mapidea__module__project=...) works but is far from generic). Maybe we use the actions instead? (but same prblem, and the ratings are not included (yet).

@MagdaN MagdaN merged commit 0208807 into master May 27, 2019
@MagdaN MagdaN deleted the rm-2019-05-newsletter branch May 27, 2019 13:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@MagdaN MagdaN

@fuzzylogic2000 fuzzylogic2000

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rmader @fuzzylogic2000 @MagdaN