search

DavidFeldhoff / al-codeactions

MIT License
17 stars 8 forks source link

Convert object ids to object names #107

Closed dannoe closed 2 years ago

dannoe commented 3 years ago

This will fix #106

I'm going to transform this to an command-palette action to fix the whole file, instead of fixing single variables.

Tips and feedback will be gladly accepted. 👍🏼

DavidFeldhoff commented 3 years ago

My first PR 🥳 Maybe we can see that as a chance to get in contact and we'll develop it together in a session? This week is quite busy with dynamicscon etc., but maybe we can catch up at the weekend. What do you think?

dannoe commented 3 years ago

I'm very busy the next time and I can't say exactly when I'll find time for it. But feel free to push changes to my branch or make comments about it.

tinfister commented 3 years ago

Looking forward to this one. Hope you'll both find time to finish the PR

dannoe commented 3 years ago

I'm sorry, but I have some bad news. @tinfister We just converted our old C/AL app completely to AL (as BaseApp) and now we use this code to copy&paste. I won't put any more work into it from my side, as I don't need the feature anymore at the moment.

@DavidFeldhoff Sorry for ruining your first PR 😞

Feel free to do whatever you want with my code.

DavidFeldhoff commented 2 years ago

I'll close this PR as Andrzej picked this one up in his extension. The command in the "AZ AL Dev Tools/AL Code Outline" extension is called "Convert Object Ids to Names in the Active Project" https://github.com/anzwdev/al-code-outline#code-transformation-commands