Skip to content

Commit 366d144

Browse files
fix: Move padding inside unverified user component (#4404)
1 parent 7be7058 commit 366d144

File tree

9 files changed

+54
-70
lines changed

9 files changed

+54
-70
lines changed
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
{{#if this.shouldShowMessage}}
2+
<div class="{{unless @padless 'mb-8'}}">
3+
<div class="ui icon {{if this.isMailSent 'info' 'error'}} message eight wide column center aligned">
4+
<i class="close icon" onclick={{action (mut this.isMessageVisible) false}} role="button"></i>
5+
<i class="info icon"></i>
6+
<div class="content">
7+
{{#if this.isMailSent}}
8+
<div class="header">
9+
{{t 'Confirmation mail has been sent again successfully!'}}
10+
</div>
11+
{{else}}
12+
<div class="header">
13+
{{#if (eq this.session.currentRouteName 'events.view.index')}}
14+
{{t 'To make your event live, please verify your account by clicking on the confirmation link that has been emailed to you.'}}
15+
{{else}}
16+
{{t 'Your account is unverified. Please verify by clicking on the confirmation link that has been emailed to you.'}}
17+
{{/if}}
18+
</div>
19+
<p>{{t 'Did not get the email?'}} <a href="#" onclick={{action 'sendConfirmationMail'}}>{{t 'Please click here to resend the confirmation mail.'}}</a></p>
20+
{{/if}}
21+
</div>
22+
</div>
23+
</div>
24+
{{/if}}

app/components/unverified-user-message.js

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,11 @@
11
import { tracked } from '@glimmer/tracking';
2-
import classic from 'ember-classic-decorator';
3-
import { action, computed } from '@ember/object';
2+
import { action } from '@ember/object';
43
import Component from '@ember/component';
54

6-
@classic
75
export default class UnverifiedUserMessage extends Component {
86
@tracked isMessageVisible = true;
97
isMailSent = false;
108

11-
@computed(
12-
'session.isAuthenticated',
13-
'authManager.currentUser.isVerified',
14-
'isMessageVisible'
15-
)
169
get shouldShowMessage() {
1710
return this.session.isAuthenticated
1811
&& this.isMessageVisible

app/styles/partials/utils.scss

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,3 +31,7 @@
3131
border-bottom-left-radius: 0 !important;
3232
border-top-left-radius: 0 !important;
3333
}
34+
35+
.mb-8 {
36+
margin-bottom: 2rem;
37+
}

app/templates/account.hbs

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,4 @@
1-
<div class="ui stackable grid">
2-
<div class="sixteen wide column">
3-
<UnverifiedUserMessage />
4-
</div>
5-
</div>
1+
<UnverifiedUserMessage @padless="true" />
62
<div class="row">
73
<div class="sixteen wide column">
84
<TabbedNavigation>

app/templates/components/unverified-user-message.hbs

Lines changed: 0 additions & 22 deletions
This file was deleted.

app/templates/create.hbs

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,5 @@
11
<div class="ui container">
2-
<div class="ui stackable grid">
3-
<div class="sixteen wide column">
4-
<UnverifiedUserMessage />
5-
</div>
6-
</div>
2+
<UnverifiedUserMessage />
73
{{#if this.model.event.name}}
84
<h2 class="weight-300">{{t 'Creating'}} <span class="weight-500">{{this.model.event.name}}</span></h2>
95
{{else}}

app/templates/events/view.hbs

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,6 @@
11
{{#if (and (or (eq this.session.currentRouteName 'events.view.index') (eq this.session.currentRouteName 'events.view.export')) (gt this.unverified-user-message.length 0))}}
22
{{#if (eq this.model.state 'draft')}}
3-
<div class="ui grid stackable">
4-
<div class="row">
5-
<div class="sixteen wide column">
6-
<UnverifiedUserMessage />
7-
</div>
8-
</div>
9-
</div>
3+
<UnverifiedUserMessage />
104
{{/if}}
115
{{/if}}
126
{{#if (not-includes this.session.currentRouteName 'events.view.edit')}}

app/templates/public.hbs

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,4 @@
1-
<div class="ui stackable grid">
2-
<div class="sixteen wide column">
3-
<UnverifiedUserMessage />
4-
</div>
5-
</div>
1+
<UnverifiedUserMessage />
62
<div class="public-event ui relaxed grid">
73
<div class="sixteen wide column lead {{if this.smallLead 'small'}}">
84
<Widgets::SafeImage @src={{if this.model.largeImageUrl this.model.largeImageUrl this.model.originalImageUrl}} />
Lines changed: 21 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,42 +1,45 @@
11
import { hbs } from 'ember-cli-htmlbars';
22
import { module, test } from 'qunit';
3-
import EmberObject from '@ember/object';
43
import { setupIntegrationTest } from 'open-event-frontend/tests/helpers/setup-integration-test';
54
import { render } from '@ember/test-helpers';
65

76
module('Integration | Component | unverified user message', function(hooks) {
87
setupIntegrationTest(hooks);
98

10-
test('it renders', async function(assert) {
11-
this.set('shouldShowMessage', true);
9+
function setShouldShowMessage(sessionExtra = {}) {
10+
this.set('session', {
11+
isAuthenticated: true,
12+
...sessionExtra
13+
});
14+
this.set('authManager', {
15+
currentUser: {
16+
isVerified: false
17+
}
18+
});
19+
}
20+
21+
test('confirmation mail sent', async function(assert) {
22+
setShouldShowMessage.call(this);
1223
this.set('isMailSent', true);
13-
await render(hbs`{{unverified-user-message shouldShowMessage=shouldShowMessage isMailSent=isMailSent}}`);
24+
await render(hbs`{{unverified-user-message isMailSent=isMailSent authManager=authManager session=session}}`);
1425
assert.ok(this.element.innerHTML.trim().includes('Confirmation mail has been sent again successfully'));
1526
});
1627

17-
test('it renders', async function(assert) {
18-
19-
let session = EmberObject.create({
28+
test('event live message', async function(assert) {
29+
setShouldShowMessage.call(this, {
2030
currentRouteName: 'events.view.index'
2131
});
22-
23-
this.set('shouldShowMessage', true);
2432
this.set('isMailSent', false);
25-
this.set('session', session);
26-
await render(hbs`{{unverified-user-message shouldShowMessage=shouldShowMessage session=session isMailSent=isMailSent}}`);
33+
await render(hbs`{{unverified-user-message session=session authManager=authManager isMailSent=isMailSent}}`);
2734
assert.ok(this.element.innerHTML.trim().includes('To make your event live, please verify your account by clicking on the confirmation link that has been emailed to you.'));
2835
});
2936

30-
test('it renders', async function(assert) {
31-
32-
let session = EmberObject.create({
37+
test('unverified message', async function(assert) {
38+
setShouldShowMessage.call(this, {
3339
currentRouteName: 'else'
3440
});
35-
36-
this.set('shouldShowMessage', true);
3741
this.set('isMailSent', false);
38-
this.set('session', session);
39-
await render(hbs`{{unverified-user-message shouldShowMessage=shouldShowMessage session=session isMailSent=isMailSent}}`);
42+
await render(hbs`{{unverified-user-message session=session authManager=authManager isMailSent=isMailSent}}`);
4043
assert.ok(this.element.innerHTML.trim().includes('Your account is unverified. Please verify by clicking on the confirmation link that has been emailed to you.'));
4144
});
4245
});

0 commit comments

Comments
 (0)