Skip to content
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

[Spike] angular frontend with ng-openapi-gen sdk #578

Closed
wants to merge 5 commits into from

Conversation

derdeka
Copy link
Contributor

@derdeka derdeka commented Feb 27, 2020

This spike aims to implement an angular frontend with the help of ng-openapi-gen.

Early feedback and contributors who like to join the spike welcomed.

Related to: loopbackio/loopback-next#4610

@hacksparrow
Copy link
Contributor

👏 excellent!

@dougal83
Copy link
Contributor

Great. What are your thoughts on hosting the angular app via express alongside the loopback application? How come you chose this particular generator did you look at others?

@derdeka
Copy link
Contributor Author

derdeka commented Feb 27, 2020

What are your thoughts on hosting the angular app via express alongside the loopback application?

I normally prefer to have seperate projects or even seperate repositories for frontend and backend as the builtin angular tooling is complex, changes frequently and built takes a while. In my projects i just deploy the generated static bundle files of the progressive webapp to aws s3 by continous deployment aws codepipeline. No (express) server needed at all.

How come you chose this particular generator did you look at others?

I did some POCs with several sdks, but none of them fits my needs. I'm just uploading here parts of my researches, ask for community feedback and still hope to find a better sdk.
If you have any other idea for an sdk we should test, please add your suggestions to the main thread loopbackio/loopback-next#4610

@dougal83
Copy link
Contributor

It would be great to have a number of deployment options demonstrated as a stretch goal. It would make loopback/angular combo very accessible.

Personally I do like the generator in the other spike as it opens the door to many other options apart from angular.

Thanks for sharing these, they are great examples.

@dhmlau
Copy link
Member

dhmlau commented Feb 29, 2020

@derdeka, thanks for looking into this!

I have two comments/questions:

  • If just comparing the angular generator, what are the differences between ng-openapi-gen and openapi-generator (you did it for [Spike] angular frontend with openapi-generator sdk #577)? Any pros and cons of using one over the other?
  • I think this spike can serve two purposes:
    1. If I create a LB4 app from scratch and now want to build a frontend, is there any generator we would recommend? We may not need to choose it for users but if one is better (in terms of ease of use, more comprehensive client, etc), we do make certain recommendation. And if we want to build an e2e tutorial, it helps us decide which one to pick.
    2. If I have a LB3 app and generated client using the Angular SDK, which path would make my migration process easier? I'm looking at the generated client in the loopback-example-angular, does it mean the *.controller.service.ts generated in this PR would map to the controllers in the /controllers folder?

@dhmlau
Copy link
Member

dhmlau commented Oct 7, 2020

FYI - I added this docs page to contain steps to create the model and web client in an angular application using ng-openapi-gen. See https://github.com/strongloop/loopback-next/blob/master/docs/site/Building-frontend-angular-application.md

@stale
Copy link

stale bot commented Dec 25, 2020

This pull request has been marked stale because it has not seen activity within two months. It will be closed within 14 days of being stale unless there is new activity.

@stale stale bot added the stale label Dec 25, 2020
@stale
Copy link

stale bot commented Jul 9, 2021

This pull request has been closed due to continued inactivity. If you are interested in finishing the proposed changes, then feel free to re-open this pull request or open a new one.

@stale stale bot closed this Jul 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants