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

Typo in notification, and suggestion for updated notification #1134

Closed
simonk83 opened this issue Feb 17, 2017 · 4 comments
Closed

Typo in notification, and suggestion for updated notification #1134

simonk83 opened this issue Feb 17, 2017 · 4 comments

Comments

@simonk83
Copy link

Ombi Version:

V 2.1.1412

Update Branch:

Early Access Preview

Operating System:

Mono (via Docker on a Synology)

Mono Version (only if your not on windows)

4.6.2 (Stable 4.6.2.16/ac9e222 Tue Jan 3 11:48:26 UTC 2017)

Applicable Logs (from /logs/ directory or the Admin page):


This series has already been added	Ombi.Core.MovieSender	Error

Problem Description:

Reproduction Steps:

I just tried to request a movie that I already have in my Radarr wanted list. The notification that popped up was "Could not add movie, please contract your administrator". There's a minor typo in there (contract instead of contact), however when I checked the log, the error in there shows as "This series has already been added". Aside from it saying series instead of movie, this seems like a much more useful message to show the user instead. "This movie is already added and pending a release" or something along those lines.

@Ombi-robot
Copy link

Hi!
Thanks for the issue report. Before a real human comes by, please make sure you used our bug report format.
Before posting make sure you also read our FAQ and known issues.
Make the title describe your issue. Having "not working" or "I get this bug" for 100 issues, isn't really helpful.
If we need more information or there is some progress we tag the issue or update the tag and keep you updated.
Cheers!
Ombi Support Team

@simonk83 simonk83 changed the title Typo in notification, and better notification suggestion Typo in notification, and suggestion for updated notification Feb 17, 2017
@Magikarplvl4
Copy link
Contributor

Magikarplvl4 commented Feb 17, 2017

This excly not a bug beceause Ombi need to show a error that it already requested. i taget this one for bug

tidusjar pushed a commit that referenced this issue Mar 10, 2017
@Magikarplvl4
Copy link
Contributor

@tidusjar you make a new typo xD

@tidusjar
Copy link
Member

HAHAHAHA

tidusjar pushed a commit that referenced this issue Mar 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants