Skip to content
This repository has been archived by the owner on Jan 25, 2023. It is now read-only.

Incorrect description for updateDateTime #166

Closed
mcupak opened this issue Apr 24, 2018 · 2 comments
Closed

Incorrect description for updateDateTime #166

mcupak opened this issue Apr 24, 2018 · 2 comments
Assignees
@mcupak @sdelatorrep
Labels
type:bug
Milestone
0.4.1

Comments

@mcupak
Copy link
Contributor

mcupak commented Apr 24, 2018

The description of the updateDateTime property is a copy-past from createDateTime.
@mcupak mcupak added this to the 0.4.1 milestone Apr 24, 2018
@mcupak mcupak self-assigned this Apr 24, 2018
@mbaudis
Copy link
Member

mbaudis commented Apr 24, 2018

I would vote for removing createDateTime. It is hard to keep this consistent & meaningful.

@sdelatorrep sdelatorrep self-assigned this Jun 11, 2018
sdelatorrep added a commit to sdelatorrep/beacon-team that referenced this issue Jun 11, 2018
@sdelatorrep
Fix wrong description (ga4gh-beacon#166).
b973421
@sdelatorrep
Copy link
Contributor

PR fixing the description: #185

Please, @mbaudis, create a new issue for discussing removing createDateTime field. Thanks!

@mbaudis mbaudis mentioned this issue Jun 11, 2018
Closed
juhtornr added a commit that referenced this issue Jun 18, 2018
@juhtornr
Merge pull request #185 from sdelatorrep/issue_166
6853b00 
Fix wrong description (#166).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mcupak mcupak

@sdelatorrep sdelatorrep

Labels
type:bug
Projects
None yet
Milestone
0.4.1
Development

No branches or pull requests
3 participants
@mbaudis @mcupak @sdelatorrep