Skip to content

add comment on updating Airtable rev-prod for new editors#1005

Open
mpadge wants to merge 4 commits intomainfrom
editor-updates
Open

add comment on updating Airtable rev-prod for new editors#1005
mpadge wants to merge 4 commits intomainfrom
editor-updates

Conversation

@mpadge
Copy link
Copy Markdown
Member

@mpadge mpadge commented Apr 20, 2026

@maelle The change to https://github.com/ropensci/dev_guide/blob/main/scripts/airtable-get-data.R is key - some editor values are NA, and these are retained and listed as editors at https://devguide.ropensci.org/softwarereview_intro.html#editorial-team. The Airtable fields are empty, just like the standard NULL ones, so I don't know where the NAs come from, but this fixes. I've also updated all Airtable records, and added extra lines to instructions on what I did there. This should fix the list.

(Oh, and we currently have only one ed on leave, Anna Krystalli, and she's cool with being unlisted there; there'll likely be another soon joining the indefinite-leave crew, but treating them as Emeritus in this context is fine.)


Checklist for dev guide maintainers, do not delete 😸

  • Review of the content in the initial language.
  • News item.
  • Translation of the content in other languages.
  • Review of the translations.

Copy link
Copy Markdown
Member

@maelle maelle left a comment

Choose a reason for hiding this comment

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

thank you!!

Comment thread scripts/airtable-get-data.R Outdated
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