jhipster / jhipster-vuejs

A Vue.js blueprint for JHipster. It will use Vue.js as the frontend library!
https://www.jhipster.tech
Apache License 2.0
301 stars 179 forks source link

Entity page: default order should be asc #596

Closed pascalgrimaud closed 4 years ago

pascalgrimaud commented 4 years ago

When you go to entity page, the entities is ordered by id + desc (reverse). But the default value is set to false See this https://github.com/jhipster/jhipster-vuejs/blob/master/generators/entity-client/templates/vue/src/main/webapp/app/entities/entity.component.ts.ejs#L25

It's related to this line of code :

const result = [this.propOrder + ',' + (this.reverse ? 'asc' : 'desc')];

Maybe tests needed to be fixed too

qmonmert commented 4 years ago

PR https://github.com/jhipster/generator-jhipster/pull/12189

cc @pascalgrimaud

qmonmert commented 4 years ago

@avdev4j you can close the ticket I think

qmonmert commented 4 years ago

@pascalgrimaud you can close the ticket I think: PR jhipster/generator-jhipster#12189