Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add method list protected ranges #1008

Merged
merged 1 commit into from
Mar 9, 2022
Merged

Conversation

lavigne958
Copy link
Collaborator

Add a new method to list all protected ranges.
This method can't be tested in our test suite because it requires to
create a new spreadsheet with a protected range. This requires us to add
the email address used to record the test interactions to create a valid
protected range. This leads to my personal service account leaked in GH.

It has been tested manually and works fine.

Closes #999

Add a new method to list all protected ranges.
This method can't be tested in our test suite because it requires to
create a new spreadsheet with a protected range. This requires us to add
the email address used to record the test interactions to create a valid
protected range. This leads to my personal service account leaked in GH.
not cool :-(

It has been tested manually and works fine.

Closes #999
@lavigne958 lavigne958 self-assigned this Mar 7, 2022
@lavigne958 lavigne958 merged commit 6cce36c into master Mar 9, 2022
@lavigne958 lavigne958 deleted the feature/list_protected_ranges branch March 10, 2022 16:20
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.

add method to list protected ranges
1 participant