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

after install have a xqerl greeter #48

Closed
5 tasks done
grantmacken opened this issue Nov 30, 2021 · 4 comments
Closed
5 tasks done

after install have a xqerl greeter #48

grantmacken opened this issue Nov 30, 2021 · 4 comments

Comments

@grantmacken
Copy link
Contributor

grantmacken commented Nov 30, 2021

  • add supervisor and server code for loading xquery main modules in priv dir
  • add main module 'greeter.xq` to priv/modules dir
  • add associated css and image files to priv/static/assets dir
  • alter cowboy routes so http://localhost:8081/xqerl will display greeter page
  • write check in actions to show greeter is working

Yep I know I could have chucked a index.html file into /priv/static/assets and let the cowboy static assets handler do its thing.
however I want to use the priv/modules dir for other stuff so I did it this way.

@grantmacken
Copy link
Contributor Author

xqerl always listens on port 8081.
so after installing xqerl there could be a greeter on http://localhost:8081
to show the install has been completed successfuly.

Prior practice: Greeters on port 80 after installing nginx or openresty.
example:

Welcome to OpenResty!!

If you see this page, the OpenResty web platform is successfully installed and working. 
Further configuration is required.

For 0nline documentation and support please refer to openresty.org.

Thank you for flying OpenResty.

https://welcome.vgs.lenovo.com.cn/

@zadean
Copy link
Owner

zadean commented Dec 1, 2021

This is a really good idea.

I think a default greeter should be overridable by a user XQuery module. Then it's there, unless it shouldn't be. right?

@grantmacken

This comment has been minimized.

This was referenced Dec 11, 2021
zadean pushed a commit that referenced this issue Dec 22, 2021
* resolve merge

* actions: typo fix
@grantmacken
Copy link
Contributor Author

Pr #52 merged

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

No branches or pull requests

2 participants