Skip to content

Update all *_owners functions to *_editors - #45

Merged
RobLBaker merged 10 commits into
DOI-NPS:mainfrom
RobLBaker:main
Aug 28, 2026
Merged

Update all *_owners functions to *_editors #45
RobLBaker merged 10 commits into
DOI-NPS:mainfrom
RobLBaker:main

Conversation

@RobLBaker

Copy link
Copy Markdown
Member

Update function names, documentation, and inner workings so that they reflect the DataStore web page fields the API call references, which are displayed as "editors" now on DataStore rather than "owners".

The intent is to make the R package and the DataStore web interface more parallel.

@RobLBaker
RobLBaker requested a review from aserrantes August 27, 2026 22:12
@RobLBaker RobLBaker self-assigned this Aug 27, 2026
@RobLBaker RobLBaker added documentation Improvements or additions to documentation enhancement New feature or request labels Aug 27, 2026
\title{Add owners to a DataStore reference}
\name{add_reference_editors}
\alias{add_reference_editors}
\title{Add editrs to a DataStore reference}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
\title{Add editrs to a DataStore reference}
\title{Add editors to a DataStore reference}

}
\description{
Add owners to a DataStore reference
Add editrs to a DataStore reference

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Add editrs to a DataStore reference
Add editors to a DataStore reference

@aserrantes aserrantes left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few small typos but otherwise looks good!

Comment thread R/set.R Outdated
Co-authored-by: Alexa Serrantes <alexaserrantes@gmail.com>
@RobLBaker
RobLBaker merged commit d2d5206 into DOI-NPS:main Aug 28, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants