Conversation
|
To cover the build failures, #231 must be merged first. |
|
@dhruvhacks Please add the newsletter models along with the migration files in a separate PR. |
- Views to access Newsletters of various departments. - Access to specific groups to add new Newsletters.
|
@RishabhJain2018 Okay sir. |
|
@RishabhJain2018 Please review #233. It contains the models and migration files of this feature. |
RishabhJain2018
left a comment
There was a problem hiding this comment.
@dhruvhacks Please add the screenshots of the UI.
|
@RishabhJain2018
|
|
@RishabhJain2018 Sir please do not merge this PR before merging #233 into this branch. This way complete feature can be tested once. Target of #233 is already set to this branch. |
|
@dhruvhacks Feel free to merge this as well as per your plan. |
|
@RishabhJain2018 Okay sir! Since this PR is stale, will update it and consider merging! |





This PR adds-
closes #229
@RishabhJain2018 please review 🙂