Skip to content

Commit

Permalink
Merge branch 'development' into side-menu
Browse files Browse the repository at this point in the history
  • Loading branch information
snitin315 authored Feb 6, 2020
2 parents 3cd51b8 + 531a8fa commit a86780e
Show file tree
Hide file tree
Showing 121 changed files with 2,992 additions and 2,168 deletions.
10 changes: 9 additions & 1 deletion .eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@ module.exports = {
sourceType: 'module'
},
plugins: [
'ember'
'ember',
'ember-suave'
],
extends: [
'eslint:recommended',
Expand Down Expand Up @@ -66,9 +67,16 @@ module.exports = {
'new-cap': 'off',
'eqeqeq': ['error', 'smart'],
'one-var': 'off',
'padding-line-between-statements': 'off',
'lines-between-class-members': ['error', "always", { exceptAfterSingleLine: true }],
'ember-suave/no-const-outside-module-scope': 'off',
'ember-suave/require-access-in-comments': 'off',
'ember-suave/lines-between-object-properties': 'off',
'ember/no-get': 'error',
// TODO: Re-enable later
'no-duplicate-imports': 'warn',
'no-await-in-loop': 'warn',
'import/no-relative-parent-imports': 'warn',
},
globals: {
module : true,
Expand Down
10 changes: 9 additions & 1 deletion .template-lintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,14 @@ module.exports = {
rules: {
'no-nested-interactive': {
ignoredTags: ['label'] // Allow label tag inside a or any other interactive element
}
},
'link-rel-noopener': true,
// TODO: Remove and fix
'require-button-type': false,
'no-partial': false,
'require-valid-alt-text': false,
'no-inline-styles': false,
'no-negated-condition': false,
'no-invalid-meta': false // Crashing the linter https://github.com/ember-template-lint/ember-template-lint/pull/1087
}
};
1 change: 1 addition & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ script:
- yarn lint:scss
- COVERAGE=true yarn test
- ROOT_URL=open-event-frontend ember build -prod
- bash scripts/test_fastboot.sh

after_success:
- 'bash <(curl -s https://codecov.io/bash)'
Expand Down
3 changes: 1 addition & 2 deletions app/adapters/application.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ import { computed } from '@ember/object';
import ENV from 'open-event-frontend/config/environment';
import JSONAPIAdapter from 'ember-data/adapters/json-api';
import HasManyQueryAdapterMixin from 'ember-data-has-many-query/mixins/rest-adapter';
import AdapterFetch from 'ember-fetch/mixins/adapter-fetch';
import FastbootAdapter from 'ember-data-storefront/mixins/fastboot-adapter';

/**
Expand All @@ -20,7 +19,7 @@ export const fixFilterQuery = query => {
return query;
};

export default JSONAPIAdapter.extend(HasManyQueryAdapterMixin, AdapterFetch, FastbootAdapter, {
export default JSONAPIAdapter.extend(HasManyQueryAdapterMixin, FastbootAdapter, {
host : ENV.APP.apiHost,
namespace : ENV.APP.apiNamespace,

Expand Down
4 changes: 2 additions & 2 deletions app/components/account/application-section.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ export default Component.extend({
this.loader.load(`/auth/oauth/login/${ provider }/${ authData.authorizationCode }/?redirect_uri=${ authData.redirectUri}`)
.then(async response => {
let credentials = {
'identification' : response.email,
'password' : response.facebook_login_hash
'username' : response.email,
'password' : response.facebook_login_hash
};

let authenticator = 'authenticator:jwt';
Expand Down
4 changes: 4 additions & 0 deletions app/components/event-invoice/billing-info.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/event-invoice/event-info.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/event-invoice/invoice-summary.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/event-invoice/payee-info.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
2 changes: 1 addition & 1 deletion app/components/forms/admin/settings/system/captcha-form.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@ import Component from '@ember/component';

export default class extends Component {

}
}
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@ import Component from '@ember/component';

export default class extends Component {

}
}
6 changes: 3 additions & 3 deletions app/components/forms/login-form.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ export default class extends Component.extend(FormMixin) {
@action
async submit() {
this.onValid(async() => {
let credentials = { identification: this.identification, password: this.password },
let credentials = { username: this.identification, password: this.password },
authenticator = 'authenticator:jwt';
this.setProperties({
errorMessage : null,
Expand All @@ -60,8 +60,8 @@ export default class extends Component.extend(FormMixin) {

}
} catch (e) {
if (e.error) {
this.set('errorMessage', this.l10n.tVar(e.error));
if (e.json && e.json.error) {
this.set('errorMessage', this.l10n.tVar(e.json.error));
} else {
this.set('errorMessage', this.l10n.t('An unexpected error occurred.'));
}
Expand Down
23 changes: 23 additions & 0 deletions app/components/forms/wizard/basic-details-step.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import { inject as service } from '@ember/service';
import EventWizardMixin from 'open-event-frontend/mixins/event-wizard';
import { protocolLessValidUrlPattern } from 'open-event-frontend/utils/validators';
import ENV from 'open-event-frontend/config/environment';
import $ from 'jquery';

export default Component.extend(FormMixin, EventWizardMixin, {

Expand Down Expand Up @@ -128,6 +129,12 @@ export default Component.extend(FormMixin, EventWizardMixin, {

// TODO: Removing the Event Time Validations due to the weird and buggy behaviour. Will be restored once a perfect solution is found. Please check issue: https://github.com/fossasia/open-event-frontend/issues/3667
getValidationRules() {
$.fn.form.settings.rules.checkMaxMinPrice = () => {
return $('.ui.form').form('get value', 'min_price') <= $('.ui.form').form('get value', 'max_price');
};
$.fn.form.settings.rules.checkMaxMinOrder = () => {
return $('.ui.form').form('get value', 'ticket_min_order') <= $('.ui.form').form('get value', 'ticket_max_order');
};

let validationRules = {
inline : true,
Expand Down Expand Up @@ -257,6 +264,10 @@ export default Component.extend(FormMixin, EventWizardMixin, {
{
type : 'number',
prompt : this.l10n.t('Invalid number')
},
{
type : 'checkMaxMinOrder',
prompt : this.l10n.t('Minimum order should not be greater than maximum')
}
]
},
Expand All @@ -274,6 +285,10 @@ export default Component.extend(FormMixin, EventWizardMixin, {
{
type : 'integer[1..]',
prompt : this.l10n.t('Maximum tickets per order should be greater than 0')
},
{
type : 'checkMaxMinOrder',
prompt : this.l10n.t('Maximum order should not be less than minimum')
}
]
},
Expand All @@ -287,6 +302,10 @@ export default Component.extend(FormMixin, EventWizardMixin, {
{
type : 'integer[1..]',
prompt : this.l10n.t('Minimum price needs to be greater than zero')
},
{
type : 'checkMaxMinPrice',
prompt : this.l10n.t('Minimum price should not be greater than maximum')
}
]
},
Expand All @@ -300,6 +319,10 @@ export default Component.extend(FormMixin, EventWizardMixin, {
{
type : 'integer[1..]',
prompt : this.l10n.t('Maximum price needs to be greater than zero')
},
{
type : 'checkMaxMinPrice',
prompt : this.l10n.t('Maximum price should not be less than minimum')
}
]
},
Expand Down
2 changes: 2 additions & 0 deletions app/components/modals/cropper-modal.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,13 @@ export default class extends ModalBase {
}

}

@action
resetImage() {
this.onHide();
this.onVisible();
}

@action
cropImage() {
$('img', this.element).croppie('result', { type: 'base64', size: 'original', quality: 1, format: 'jpeg' }).then(result => {
Expand Down
4 changes: 4 additions & 0 deletions app/components/orders/organizer-info.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
2 changes: 1 addition & 1 deletion app/components/paypal-button.js
Original file line number Diff line number Diff line change
Expand Up @@ -136,4 +136,4 @@ export default Component.extend({

}
}
});
});
3 changes: 1 addition & 2 deletions app/components/public/ticket-list.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,8 @@ import Component from '@ember/component';
import { computed } from '@ember/object';
import FormMixin from 'open-event-frontend/mixins/form';
import { inject as service } from '@ember/service';
import { sumBy } from 'lodash-es';
import { sumBy, merge } from 'lodash-es';
import { A } from '@ember/array';
import { merge } from 'lodash-es';

export default Component.extend(FormMixin, {
store: service(),
Expand Down
1 change: 1 addition & 0 deletions app/components/tables/utilities/pagination.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ export default class extends Component {
return this.currentPage <= 1;

}

@computed('currentPage', 'pageCount')
get moveToNextPageDisabled() {
return this.currentPage >= this.pageCount;
Expand Down
3 changes: 1 addition & 2 deletions app/components/ui-table-server.js
Original file line number Diff line number Diff line change
Expand Up @@ -83,8 +83,7 @@ export default ModelsTable.extend({
if (!get(this.data, 'query')) {
console.warn('You must use https://emberjs.com/api/data/classes/DS.Store.html#method_query for loading data');
query = merge({}, this.query);
store = this.store;
modelName = this.modelName;
({ store, modelName } = this);
} else {
query = merge({}, get(this.data, 'query'));
store = get(this.data, 'store');
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/cell/admin/messages/cell-options.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/cell/cell-event-date.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/cell/cell-event-state.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/cell/cell-sessions-dashboard.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/cell/cell-speakers-dashboard.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/cell/cell-sponsor-image.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/cell/cell-sponsor-sanitize.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/cell/events/cell-action.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/cell/events/cell-amount.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/cell/events/cell-event-invoice.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/expand-row-cell.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/header-row-filtering.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/header-row-sorting.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/header-rows-grouped.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
4 changes: 4 additions & 0 deletions app/components/ui-table/header-sorting-icons.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
import Component from '@ember/component';

export default Component.extend({
});
2 changes: 1 addition & 1 deletion app/components/widgets/forms/billing-info.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@ export default Component.extend({
countries: computed(function() {
return orderBy(countries, 'name');
})
});
});
2 changes: 1 addition & 1 deletion app/controllers/account/billing/invoices/list.js
Original file line number Diff line number Diff line change
Expand Up @@ -139,4 +139,4 @@ export default class extends Controller.extend(EmberTableControllerMixin) {
}
return columns;
}
}
}
Loading

0 comments on commit a86780e

Please sign in to comment.