Skip to content

Fixed Day (was getting Month)#2

Open
kdaily wants to merge 1 commit intoskoval:masterfrom
kdaily:fixday
Open

Fixed Day (was getting Month)#2
kdaily wants to merge 1 commit intoskoval:masterfrom
kdaily:fixday

Conversation

@kdaily
Copy link
Copy Markdown

@kdaily kdaily commented Feb 16, 2015

I applied this on cran/RISmed too; not sure where is the proper place!

@skoval
Copy link
Copy Markdown
Owner

skoval commented Feb 19, 2015

Hey, Thanks for pointing out the day problem. Your correction has been incorporated into the current version.

I also did a fairly extensive revamp of the date variables to reflect more aspects of the publication history (accepted, received, ppublish, etc.) If you could install the package from the repo, and let me know what you think about the date methods (Year, Month, etc.) that would be great. One problem is that the different parts of the history are not always consistently populated so there are usually NAs in any particular history field.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants