Skip to content
This repository has been archived by the owner on May 29, 2023. It is now read-only.

fix: preserve field names in the first row #1089

Merged
merged 2 commits into from
May 20, 2022
Merged

fix: preserve field names in the first row #1089

merged 2 commits into from
May 20, 2022

Conversation

tmatsuo
Copy link
Contributor

@tmatsuo tmatsuo commented May 20, 2022

fixes #1087

This is still ugly, but it won't break the internal datastudio, I think.

fixes #1087

This is still ugly, but it won't break the internal datastudio, I think.
@tmatsuo tmatsuo requested a review from a team as a code owner May 20, 2022 18:28
@tmatsuo tmatsuo requested a review from a team May 20, 2022 18:28
@product-auto-label product-auto-label bot added the size: s Pull request size is small. label May 20, 2022
@@ -167,10 +167,11 @@ export async function exportApisToSheets() {
const values = await getResults();
values.unshift(['Service', 'Title', 'Group', 'HasSurface', 'InScope', 'ToS']);
Copy link
Contributor

Choose a reason for hiding this comment

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

won't this lead to the first column being added multiple times, since we're no longer clearing A1?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it will override

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size: s Pull request size is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

A moment of nothing
2 participants