-
Notifications
You must be signed in to change notification settings - Fork 97
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 documentation for RequiredWith migration from SDKv2 to plugin framework #657
Comments
Hi @remyleone 👋 Thank you for raising this. If you have any specific suggestions, please let us know. There's currently the SDK to framework migration guide, but once it gets to individual But it doesn't say the exact solutions available for that particular field:
@bendbennett do you think we should have https://developer.hashicorp.com/terraform/plugin/framework/migrating/attributes-blocks/fields and/or https://developer.hashicorp.com/terraform/plugin/framework/migrating/attributes-blocks/validators-predefined be a little bit more prescriptive about the mapping between the old and new? It may be useful to have tables, such as::
etc. |
It would be advantageous to have a table that lists each field and how you should translate it into the framework. I've been looking around many pages to see how I should try out to translate a given pattern. In particular, having a linter tool that offers automated suggestions/helps even if it is very simplistic would help give confidence that it could be achieved. While waiting for an automated tool to appear a table would be very much appreciated and offer great help to map possible migration. |
@bflad I agree that more prescriptive mapping would be useful. To this end I've added an issue for Add more prescriptive documentation for mapping between SDKv2 and the Framework and opened a related PR for Documentation for mapping between SDKv2 and the Framework. I'm happy to expand on this further if additional more prescriptive documentation is useful or desired. |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Module version
Use-cases
Attempted Solutions
Proposal
References
The text was updated successfully, but these errors were encountered: