JOSS Review warning about out of date local data

[DrMattG]

Warning message:
In wdpa_fetch("Norway", wait = TRUE, download_dir = rappdirs::user_data_dir("wdpar")) :
local data is out of date: jul 2022
(

wdpar/R/wdpa_fetch.R

Line 183 in 40e8e79

### throw warning if out of date

)
What does it mean if the local data is out of date? Do I need to worry about this warning? It would be good to have some comment on this in the documentation (or somewhere).
openjournals/joss-reviews#4594

[jeffreyhanson]

The package will, by default, load the latest version of the database that is available on your computer (e.g., because you previously downloaded the data). This warning message is telling you that this version is now out of date, and a newer version is available online. I guess you would need to worry about this warning if you are running analyses that require the latest version, but generally this isn't the case. Sure, I can add some info to the docs on this message.

[jeffreyhanson]

Sorry for the slow progress on this @DrMattG. Just to keep you in the loop, I'm planning on creating a PR to incorperate this feedback - once I've merged the PR to address the issues raised by @Jo-Schie. Since I'm really bad at dealing with merge conflicts, I'd rather not create multiple branches at the same time and then have to juggle conflicts when merging them. Did you have any other issues/concerns (apart from #58 and #59) as part of your review? Or will I have addressed all your major concerns once these are addressed? Thanks for your patience!

[DrMattG]

Hi @jeffreyhanson - That's fine - I think I am happy with everything else - I understand the merge conflict issue!

[jeffreyhanson]

Ok awesome - thank you!

[jeffreyhanson]

@DrMattG, I've just created a PR to address this issue. When you get a chance, could you please take a look? I've put the changes specifically related to this issue in this commit: c83514c.

[jeffreyhanson]

Woops, forgot tag this issue in the commit when merging this PR, so I'll close this manually.