KartanHQ / intellij-nestjs

Nest plugin for IntelliJ Platform
Apache License 2.0
12 stars 2 forks source link

Feature/nestjs cli menu #243

Closed dinbtechit closed 1 year ago

dinbtechit commented 1 year ago

Feature #213

  1. Now we can right click anywhere in the project to invoke nest cli

image

  1. Select a type to generate image

  2. provide a filename and options, click ok to generate image

  3. Generated using Nest intellij Plugin image

dinbtechit commented 1 year ago

@nekofar - Hopefully no more changes and its ready for review now.

Some notes for future improvements:

  1. Identify if it is worth in moving the label texts to the message bundle. If so, we might want to consider refactoring that.
  2. Unit Tests / E2E Test case to cover all scenarios. (At the moment, I haven't explored much with unit testing IntelliJ plugins.)
dinbtechit commented 1 year ago

Few more updates:

https://github.com/KartanHQ/intellij-nestjs/assets/17984781/d4ef08fb-c655-4b73-902e-669d5115f78e

https://github.com/KartanHQ/intellij-nestjs/assets/17984781/e56586fc-f5c1-4577-b1ef-a746fa8f1526

dinbtechit commented 1 year ago

@nekofar - Hi Mate! Hope you doing great. Just thought I would just follow up on this PR to see if you have any concerns with the code or something. Cheers!

nekofar commented 1 year ago

@nekofar - Hi Mate! Hope you doing great. Just thought I would just follow up on this PR to see if you have any concerns with the code or something. Cheers!

Hey there! Apologies for the delay in reviewing your work. My bandwidth has been a bit limited lately, but I'm making it a priority to get to it soon. I just need to make a few changes to the repository before we can merge it. Thanks for your patience! 💚

dinbtechit commented 1 year ago

@nekofar - Oh man. I totally hear you. I was in the same boat a few months ago. Please let me know if you would like me to create this as a separate plugin if it is too much for you to take on at the moment. I also have plans to enhance IntelliSense and generate automated test cases in the future. I was hoping to do some more PRs for them. But I dont want to bog you down with too many requests. Cheers!