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

Add Quarkus multi file script example. #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pjgg
Copy link

@pjgg pjgg commented Feb 12, 2021

Quarkus Cryptocurrency example:

Features tested:

  • Multi-file script
  • External file configuration
  • Dependencies version from Bom
  • Quarkus/vertx
  • OpenAPI

@pjgg pjgg changed the title Example: quarkus multi file script Add Quarkus multi file script example. Feb 12, 2021
@maxandersen
Copy link
Collaborator

its nice with a multi file example but i'm not sure user will know what to do.. localhost:9090 just shows error and non of the /api/ end points seem to do anything.

Some basic docs/info - maybe in form of a embedded index.html ?

@pjgg
Copy link
Author

pjgg commented Feb 18, 2021

So I've pushed an Html page that uses the script endpoints.
With this example, the end-user just have to run the main script:

jbang stockCryptoCurrency.java

Then go to http://localhost:9090 on their browser, and from there they will load all cryptocurrency names (by default they don't have to do anything)... this is the heaviest part from a network point of view. Then, they could click over the names and get the current stock market value....in Euros, Bitcoins, and US dollars.

Note: Also this command should work

jbang https://github.com/pjgg/jbang-examples/blob/example/quarkus-multifile/examples/quarkus-cryptocurrency/stockCryptoCurrency.java

Amazing!!! @maxandersen

@maxandersen
Copy link
Collaborator

i missed this one - does the example actually work for you now ? I'm wondering if something broke on jbang side or the script itself?

index.html just says "Loading...." for me ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants