martyr-deepin / deepin-screenshot

Deprecated (migrated into deepin-screen-recorder after v20), consider create and maintain your own fork if you love it.
https://www.deepin.org/original/deepin-screenshot/
GNU General Public License v3.0
347 stars 102 forks source link

Notifications are still sent despite --no-notification #22

Open bfayers opened 6 years ago

bfayers commented 6 years ago

Notifcations are still sent despite running with -n or --no-notification

char commented 6 years ago

When running with -n and -s <path>, "file save" notifications are still sent.

bfayers commented 6 years ago

Yes, this is the issue i'm running into.

phocean commented 6 years ago

Please merge the patch, it is very annoying.

char commented 6 years ago

@phocean I don't think the deepin team actually check GitHub for pull requests etc.

hualet commented 5 years ago

Sorry for not paid enough attention to the Github since we used Gerrit for code review. But we're now all working on Github, so can you update your PR @half-cambodian-hacker-man, it's conflicting with other commits.

mblesel commented 5 years ago

60