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
rewrite profile about me in react with i18next #1005
Conversation
58b5198
to
2a5f605
Compare
{(!profile.description || profile.description === '') && | ||
<blockquote className="profile-quote" | ||
aria-label={t('Member has not written description about themself.')}> | ||
“Everyone is necessarily the hero of their own life story.” |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's simplify this and just not show the whole box if they didn't write anything.
We'll still need to show the area + edit link if isSelf
is true.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh and again good for a follow up PR, not a blocker here.
}; | ||
|
||
export const plainTextLength = (text) => { | ||
return text && typeof(text) === 'string' ? String(text).replace(/ /g, ' ').replace(/<[^>]+>/gm, '').trim().length : 0; // eslint-disable-line angular/typecheck-string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eslint-disable-line angular/typecheck-string
Hmm, it's surprising that Angular linter is trying to lint here.
No need to change anything, I'm just noting to look into this when working again on eslint stuff: #932
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall I like what I read and I think we'll be able to merge soon.
From my perspective there is one bug that needs to be fixed and a few cleanups that can be fixed, and linting.
The bug and linting and I'll be ready to approve this for merge.
👍
modules/users/client/views/profile/profile-view-about.client.view.html
Outdated
Show resolved
Hide resolved
modules/users/client/views/profile/profile-view-about.client.view.html
Outdated
Show resolved
Hide resolved
modules/users/client/views/profile/profile-view-about.client.view.html
Outdated
Show resolved
Hide resolved
4a609c2
to
5a1fd14
Compare
5a1fd14
to
c9e2d5f
Compare
I believe this is ready for merge. I just rebased with master and changed the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't tested but looks good!
Merge at will. :-)
Proposed Changes
Testing Instructions
/profile/:username
and observe that everything works the same as before.i18n
translations should work if data for translations are added topublic/locales/:languagecode
files