ui5-community / generator-ui5-project

Generator for UI5-based web-apps which use the official UI5 tooling and support multiple deployment targets such as the SAP Business Technology Platform
Apache License 2.0
33 stars 25 forks source link

Feature: FLP Plugin Project Type #11

Closed ubheamar closed 3 years ago

ubheamar commented 3 years ago

Hello Marius,

I was thinking to create a generator for the FLP plugin project. Can you suggest where should I add this generator generator-ui5-project or generator-ui5-library or entirely create a new project in UI5 Community

I saw the main generator generator-easy-ui5 fetching repository from UI5 Community & showing list if we don't pass any options to the generator.

Let me know any other contribution required from my side I am happy to do that.

IObert commented 3 years ago

Hi Amar,

thanks for opening this issue. You are right, easy-ui5 pulls the templates from this GitHub org.

My first reaction would be that it's probably easiest to fork this project, remove files that you don't need and add the plugin-specific details.

CC: @petermuessig WDYT?

petermuessig commented 3 years ago

Hi @ubheamar ,

nice to see you here! 👍

Regarding the FLP plugin: if it should be a standalone project which just creates the FLP plugin project, it makes sense IMO to have a separate generator. But if you intend to make this a flavour of a UI5 project - to add a FLP plugin there then it maybe belongs better to the ui5-project.

@IObert : we should also discuss how to externalize the platform specific enablement you have in the ui5-project generator. This can be maybe useful for different generators as an addition.

Best regards, Peter

IObert commented 3 years ago

how to externalize the platform specific enablement

Yes, I was thinking along the same lines. But I'm not sure if we can do this. Then entire project is more or less just the platform specific enablement + the subgenerators (e.g. for models etc). Some subgenerators (e.g. for views) are even used for the main generator. So I'm not sure if it make much sense to split this project even further.

ubheamar commented 3 years ago

Thanks for the reply 👍 @IObert & @petermuessig.

I think we already have sufficient generators for generator-ui5-project. it's better to not split more. Also for the FLPPlugin generator, There will be entirely different manifest.json structure & project structures, I thinking to fork this project build a standalone generator similar to generator-ui5-project

Give me your comments so that I can start working on it.

Regards, Amar Ubhe

petermuessig commented 3 years ago

One you are ready, let us know so that we can create a project for you

ubheamar commented 3 years ago

Hello @IObert & @petermuessig, I am ready with the project. can you create a project for me so that I can upload my code? Regards, Amar Ubhe

IObert commented 3 years ago

How do you want to name this project? And would you prefer an fresh project or should I create a fork of ui5-project (so that you could merge upstream changes every now and then)?

ubheamar commented 3 years ago

I think fresh project will be good. I have given the app name generator-ui5-flp-plugin.

IObert commented 3 years ago

Alright, I created a (for now) private repo for you to test. And I also assigned your the "write" access.

@petermuessig Is this ok or do you usually give "maintain" access out?

IObert commented 3 years ago

https://github.com/ui5-community/generator-ui5-flp-plugin

petermuessig commented 3 years ago

Hi @IObert , yes - we give "maintain" access unless there are some issues in future. I am a big fan of trust... 😄

petermuessig commented 3 years ago

You could finally close this issue, right? The new project has been created...

ubheamar commented 3 years ago

Hi @IObert , I have uploaded the code & tested it from my side as well. it's working fine with the easy-ui5 generator. If any of you test & provide feedback that will be great. I will close this issue. Also, if I have to communicate with you guys in the future, should I create an issue for that new repository or set up a discussion tab? what will be further steps to make the repo public?

IObert commented 3 years ago

I tested it and I'll provide feedback via issues in the new repo, ok?

Yes, I guess opening issues for other repos is a good approach.