Skip to content

mv 'airtable.png' -> 'airtable-filters.png' for #1009#1010

Merged
maelle merged 12 commits into
mainfrom
airtable
Jun 25, 2026
Merged

mv 'airtable.png' -> 'airtable-filters.png' for #1009#1010
maelle merged 12 commits into
mainfrom
airtable

Conversation

@mpadge

@mpadge mpadge commented May 18, 2026

Copy link
Copy Markdown
Member

@joelnitta Thanks for reporting #1009. I'm not sure what your view looked like, and have of course 😆 no way of finding out. The Airtable link should lead you straight to the new image added in this PR:

airtable

What it shouldn't look like is this:

image

Can you please confirm which one you see?


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.

@mpadge mpadge requested review from a team as code owners May 18, 2026 13:57
@mpadge mpadge requested review from fblpalmeira and yabellini and removed request for a team May 18, 2026 13:57
@joelnitta

joelnitta commented May 19, 2026

Copy link
Copy Markdown
Contributor

I see the second one ("what it shouldn't look like").

I figured out why: I accessed AirTable via the email invite link I received when I became an editor. After more searching, I finally found the text in the Dev Guide section 9.3 that describes how to access the link going to the other view ("linked in the description of the editors-only channel on Slack"). It's a bit subtle/buried, you may want to make that clearer in other places that mention AirTable as well, probably in the filtering explanation referenced in #1009 at a minimum.

@fblpalmeira fblpalmeira left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Done @mpadge

Comment thread softwarereview_editor.pt.Rmd Outdated
Comment thread softwarereview_editor.pt.Rmd Outdated
Comment thread softwarereview_editor.pt.Rmd Outdated
maelle and others added 3 commits June 1, 2026 08:44
Co-authored-by: Francesca Palmeira <francesca@alumni.usp.br>
Co-authored-by: Francesca Palmeira <francesca@alumni.usp.br>
Co-authored-by: Francesca Palmeira <francesca@alumni.usp.br>
@maelle

maelle commented Jun 1, 2026

Copy link
Copy Markdown
Member

Thank you @fblpalmeira!

@yabellini friendly reminder 😸

@yabellini yabellini left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I did some changes, but I don't understand why there is different content in each language. English has more content changes in this PR than Spanish.

Comment thread softwarereview_editor.es.Rmd Outdated
Co-authored-by: Yanina Bellini Saibene <yabellini@gmail.com>
@maelle

maelle commented Jun 12, 2026

Copy link
Copy Markdown
Member

Ouch, thanks so much for catching this @yabellini

@maelle

maelle commented Jun 12, 2026

Copy link
Copy Markdown
Member

@mpadge just to clarify for future PRs, was this a PR where you had added translations from the get-go?

Comment thread softwarereview_editor.es.Rmd Outdated
Comment thread softwarereview_editor.es.Rmd Outdated

@yabellini yabellini left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you @maelle ! Now it is done.

@mpadge

mpadge commented Jun 13, 2026

Copy link
Copy Markdown
Member Author

@mpadge just to clarify for future PRs, was this a PR where you had added translations from the get-go?

Yes, it was

Co-authored-by: Yanina Bellini Saibene <yabellini@gmail.com>
Comment thread softwarereview_editor.pt.Rmd Outdated
Comment thread softwarereview_editor.pt.Rmd Outdated
maelle and others added 2 commits June 25, 2026 13:21
Co-authored-by: Beatriz Milz <42153618+beatrizmilz@users.noreply.github.com>
Co-authored-by: Beatriz Milz <42153618+beatrizmilz@users.noreply.github.com>
@maelle

maelle commented Jun 25, 2026

Copy link
Copy Markdown
Member

Thank you @beatrizmilz

@maelle maelle merged commit 2e65fc0 into main Jun 25, 2026
1 check passed
@maelle maelle deleted the airtable branch June 25, 2026 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants