Closed Bug 561824 Opened 10 years ago Closed 8 years ago

'Article Edit' approval notification email does not contain locale in the article link

Categories

(support.mozilla.org :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: vish_moz, Unassigned)

Details

STR:
1. log into support stage (non en-US locale) like <https://support-stage-new.mozilla.com/es> with account 'A'
2. make 'edit' to an article <https://support-stage-new.mozilla.com/es/kb/C%C3%B3mo%20limpiar%20el%20cach%C3%A9>
3. log-in with an 'Approver' account 'B'
4. Approve the edit made to the article in step 2
5. Check the email of Account A for the approval notification

Actual result:
The article link in the email is <https://support-stage-new.mozilla.com/en-US/kb/C%C3%B3mo+limpiar+el+cach%C3%A9>

Expected result:
The link should be <https://support-stage-new.mozilla.com/es/kb/C%C3%B3mo+limpiar+el+cach%C3%A9>
Vishal, let's work on finding out if this is only approval notifications.
Keywords: qawanted
This behavior is seen for all types of notifications (Category/Page/Locale).
This is also happening on Prod so its not a regression from current release.
Changing to 1.5.5
Target Milestone: 1.5.4 → 1.5.5
Keywords: qawanted
QA Contact: kb-articles → stephen.donner
Got time to look into this?
Assignee: nobody → paulc
Priority: -- → P1
Looking at the code, it seems that the locale of the email is based on the locale of the user's preferences, not the URL, which is how it should be, IMO.

I verified that this is the case on stage-new for approval notifications.

Vishal/Stephen: you could play with the user's locale and see if that changes the notification's locale. Reopen this if necessary.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
For clarification, here are my STR:

1. Login with user A, make sure your "Language" setting in tiki-user_preferences.php is NOT 'en-US', say 'ro'
2. Edit a staging article.
3. Log in with a different user B. Approve that edit.

Expected/Actual:
User A receives an email with the article url like so:
https://su.mo/ro/kb/Article+name

Haven't tested all notifications but even if locale is not set, that's fine - if user is logged in, the redirect is to the user's locale.
Sorry for the confusion. I was wrong in the STR in comment 0. The article link int he approval email does not have any locale. So, when you click on the link it redirects you to en-US even though you have set your "language' preference to other language.
This is the email I got:
The page new_test was changed by vish_moz at Wed 09 of Jun, 2010 15:02 PST
Comment: aprroved

You can view the page by following this link:
https://support-stage-new.mozilla.com/kb/new_test?bl=n

Reopening...
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Summary: 'Article Edit' approval notification email provides wrong locale link → 'Article Edit' approval notification email does not contain locale in the article link
Talked about this - if you are logged in, you will be redirected to your own locale. However, for logged out users receiving this notification, they will be taken to their browser's accept-language, which is not expected behavior.

The bug is legitimate but I'm unprioritizing it. James, feel free to push it out or take the bug if you need it within 3 weeks :)
Priority: P1 → --
Assignee: paulc → nobody
Component: Knowledge Base Articles → General
QA Contact: stephen.donner → general
We have fixed this a few weeks ago.
Status: REOPENED → RESOLVED
Closed: 10 years ago8 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.