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

feat: add comment to routeros_wifi resource #455

Merged
merged 1 commit into from
May 14, 2024

Conversation

redcancode
Copy link
Contributor

I wanted to manage the wifi interfaces on my router and I noticed that the comment property isn't part of the routeros_wifi resource schema. So I thought I add it. I've tested it by locally building the binary and running it with terraform, everything seemed to worked correctly.

This is my first contribution to this repository. I haven't found any rules in the repo so I hope this pull request is okay. Let me know if I should change something or add something.

@redcancode redcancode requested a review from a team as a code owner May 13, 2024 21:35
@vaerh
Copy link
Collaborator

vaerh commented May 14, 2024

Everything is perfect. Thanks for the PR. The only point, the generation of documentation is automatic with each release. You may not have to make changes to these files in the future.

@vaerh vaerh merged commit 5a2782c into terraform-routeros:main May 14, 2024
3 checks passed
@vaerh
Copy link
Collaborator

vaerh commented May 14, 2024

🎉 This PR is included in version 1.48.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@vaerh vaerh added the released label May 14, 2024
@redcancode
Copy link
Contributor Author

Thanks for the merge and release. 🥹 I didn't know about the generated documentation, I will watch out for that, thanks. 👍🏻

@redcancode redcancode deleted the add-comment-to-wifi branch May 14, 2024 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants