Add QuickFix links for missing configs and translations#443
Conversation
|
hey @N1ebieski, This is a great feature. Thanks for working on it! For environment variables, we currently have a "quick fix" in place:
Would it be possible to add something similar for translations, instead of linking to them from the hover card? |
…osticcode-for-diagnosticProvider-#55
…for-diagnosticProvider-#55' of https://github.com/N1ebieski/vs-code-extension into Improvements-for-linked-parameter-vscodeDiagnosticcode-for-diagnosticProvider-#55
…rovements-for-linked-parameter-vscodeDiagnosticcode-for-diagnosticProvider-#55
|
@TitasGailius This wasn't easy to achieve... Needs more testing. Contains #513 |
…r diagnosticProvider
|
Changes required according to: #513 (comment) |
|
@TitasGailius ok, summary:
|
|
@N1ebieski this is much simpler now. Thanks. One thought: since we already have all available config paths in For example, for
And if none match, we skip the code action. |
|
@TitasGailius We can do this for configs, but probably not for translations, because they only contain final keys. Did you check this? |
|
@N1ebieski you're right. We can do this for configs, but not for translations. |
|
@TitasGailius OK, let me know if I understand this correctly. Let’s assume we have a return [
'feature' => [
'item' => 'value'
]
];The user wants to access
As a result, it adds the following to the file: return [
'new-feature' => [
'new-item' => ''
],
'feature' => [
'item' => 'value'
]
]; |
|
@N1ebieski yep, I think that would cover all cases. Unless you have a better idea. 🙂 |
|
@TitasGailius another summary:
|
…nked-parameter-vscodeDiagnosticcode-for-diagnosticProvider-#55
…nked-parameter-vscodeDiagnosticcode-for-diagnosticProvider-#55




This PR improves my previous PR #415