Fix data.remote R check binding notes - #4013

Description

Addresses part of #2758 by cleaning up PEcAn.data.remote R check binding notes. Per review feedback, this removes the unused grid2netcdf() helper rather than trying to repair it.

• Removes unused grid2netcdf() and its generated Rd file.
• Keeps the cleaner foreach zip loop in download.LandTrendr.AGB().
• Updates CHANGELOG.md, modules/data.remote/NEWS.md, and the modules.data.remote reference check log.

Motivation and Context

This reduces the R CMD check notes tracked in #2758 for the PEcAn.data.remote package.

Fixes part of #2758.

Review Time Estimate

• Immediately
• Within one week
• When possible

Types of changes

• Bug fix (non-breaking change which fixes an issue) Fix tidyverse "no visible binding for global variable" notes #2758
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My change requires a change to the documentation.
• My name is in the list of CITATION.cff
• I agree that PEcAn Project may distribute my contribution under any or all of
  • the same license as the existing code,
  • and/or the BSD 3-clause license.
• I have updated the CHANGELOG.md.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

Validation

• git diff --check
• Parsed the edited R files with Rscript
• Ran a targeted codetools usage check for download.LandTrendr.AGB()

R CMD check --no-manual --no-build-vignettes modules/data.remote was attempted locally, but could not complete because this fresh environment is missing the package dependency stack (curl, DBI, ncdf4, PEcAn.*, etc.).