Skip to content
This repository has been archived by the owner on Mar 21, 2021. It is now read-only.

Map all dates from server in HTTP services #482

Open
1 task done
hdurix opened this issue Oct 15, 2019 · 3 comments · May be fixed by #632
Open
1 task done

Map all dates from server in HTTP services #482

hdurix opened this issue Oct 15, 2019 · 3 comments · May be fixed by #632
Labels
$$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ enhancement New feature or request $100 https://www.jhipster.tech/bug-bounties/

Comments

@hdurix
Copy link
Member

hdurix commented Oct 15, 2019

Overview of the feature request

It would be better to map every dates we receive from the server to Date objects directly in services (between axios and components).

Motivation for or Use Case

We could use dates as real dates and not sometimes as dates and sometimes as strings.
See for instance how it is handled in an Angular app: https://github.com/jhipster/jhipster-sample-app/blob/master/src/main/webapp/app/entities/operation/operation.service.ts#L60-L65.

The only difference is that we use plain Date in Vue.js instead of moment types in Angular.

We should do this in entities and in admin pages (user & audits).

The Date filter may also change in order to handle date values instead of strings.

Related issues or PR

#479
#464

  • Checking this box is mandatory (this is just to show you read everything)
@pascalgrimaud pascalgrimaud added $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ $100 https://www.jhipster.tech/bug-bounties/ enhancement New feature or request labels Oct 15, 2019
@snitin315
Copy link
Contributor

@hdurix @pascalgrimaud Can I take this up?

@atomfrede
Copy link
Member

@snitin315 Sure!

@pascalgrimaud
Copy link
Member

yes plz @snitin315, go ahead :)

snitin315 added a commit to snitin315/jhipster-vuejs that referenced this issue Jun 16, 2020
@snitin315 snitin315 linked a pull request Jun 16, 2020 that will close this issue
4 tasks
snitin315 added a commit to snitin315/jhipster-vuejs that referenced this issue Jun 16, 2020
snitin315 added a commit to snitin315/jhipster-vuejs that referenced this issue Jun 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
$$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ enhancement New feature or request $100 https://www.jhipster.tech/bug-bounties/
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants