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

954 duplicate notifications #112

Merged
merged 3 commits into from
Mar 4, 2016
Merged

Conversation

jasonmule
Copy link
Contributor

This pull request makes the following changes:

  • Filters contacts and notifications using the currently logged in user

Test these changes by:

  • Log in as a user and navigate to Notfications. You should only see contacts that you own even if you are an admin user.

Fixes ushahidi/platform#954 .

Ping @ushahidi/platform

Review on Reviewable

@rjmackay rjmackay deployed to ushahidi-platform-maste-pr-112 March 4, 2016 03:17 Active
@rjmackay rjmackay deployed to ushahidi-platform-maste-pr-112 March 4, 2016 19:00 Active
@rjmackay
Copy link
Contributor

rjmackay commented Mar 4, 2016

@jasonmule Added another minor fix to this branch - "Add notification" buttons weren't showing up on the saved search pages.

This is good to go now.

@jasonmule
Copy link
Contributor Author

👍 Thanks!

rjmackay added a commit that referenced this pull request Mar 4, 2016
@rjmackay rjmackay merged commit eab3822 into develop Mar 4, 2016
@rjmackay rjmackay deleted the 954-duplicate-notifications branch March 22, 2016 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants