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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

document firewall_id instance resource attribute #1234

Merged
merged 2 commits into from
Jan 19, 2024

Conversation

pbohman
Copy link
Contributor

@pbohman pbohman commented Jan 12, 2024

馃摑 Description

What does this PR do and why is this change necessary?

Documents the firewall_id instance resource attribute

What are the steps to reproduce the issue or verify the changes?

View the firewall_id instance resource attribute documentation

@pbohman pbohman requested a review from a team as a code owner January 12, 2024 16:15
@pbohman pbohman requested review from yec-akamai and vshanthe and removed request for a team January 12, 2024 16:15
Copy link
Contributor

@yec-akamai yec-akamai left a comment

Choose a reason for hiding this comment

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

@pbohman Thank you for the contribution!

@yec-akamai yec-akamai requested review from a team, jriddle-linode and zliang-akamai and removed request for a team January 19, 2024 17:37
Copy link
Member

@zliang-akamai zliang-akamai left a comment

Choose a reason for hiding this comment

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

LGTM!

@zliang-akamai zliang-akamai merged commit c075cd7 into linode:dev Jan 19, 2024
4 checks passed
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.

None yet

3 participants