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 a mechanism for custom message in home app (for demo server) #184

Closed
superstar54 opened this issue Oct 23, 2024 · 8 comments · Fixed by #183
Closed

Add a mechanism for custom message in home app (for demo server) #184

superstar54 opened this issue Oct 23, 2024 · 8 comments · Fixed by #183
Assignees

Comments

@superstar54
Copy link
Member

superstar54 commented Oct 23, 2024

Add demo message (message saying that we are in a demo, how long data is kept, …) in AiiDAlab home (initial page with ‘File manager’ ‘Terminal’ … QEapp logo [Jason and Edan] [copied above]
Make sure the default guide/tutorial is on, in the demo server. [Edan Jason Xing]

@giovannipizzi
Copy link
Member

We discussed to just have a generic functionality to show a warning message in a yellow box if a file with given name is in a given location (e.g. ~/.aiidalab/warning.md).

This is very general, then in the AiiDAlab demo deployment, we make sure that the file is present in that location.

@giovannipizzi
Copy link
Member

The content of the md should be:

Welcome to the AiiDAlab demo server!

IMPORTANT DISCLAIMER: This server has limited resources and is intended for demonstration purposes only.
The server is reset to its initial state every few days, removing all user data. Therefore, do not keep any important files or data on the server.

If this is your first time here, you can start by submitting a new Quantum ESPRESSO calculation using the buttons below!

@cpignedoli
Copy link
Member

I added a recommendation for the "fast" protocol.

Welcome to the AiiDAlab demo server!

IMPORTANT DISCLAIMER: This server has limited resources and is intended for demonstration purposes only. The server is reset to its initial state every few days, erasing all user data. Please do not store any important files or data here.

The available resources per user are optimized for simple bulk calculations using the "fast" protocol. More complex simulations may take a long time and could block your user instance.

If this is your first visit, you can get started by submitting a new Quantum ESPRESSO calculation using the buttons below!

@edan-bainglass edan-bainglass transferred this issue from aiidalab/aiidalab-qe Nov 21, 2024
@danielhollas danielhollas changed the title Add more demo server message Add a mechanism for custom message in home app (for demo server) Nov 21, 2024
@danielhollas
Copy link
Contributor

If this is your first visit, you can get started by submitting a new Quantum ESPRESSO calculation using the buttons below!

To clarify (for the purposes of Edan's PR), this is meant to be displayed in the home app, correct? "buttons below" in this context sounds a bit vague (since users still need to open the QeApp?).

@edan
Copy link

edan commented Nov 21, 2024

Hi! I think you have been tagging the wrong Edan. Cool project though!

@danielhollas
Copy link
Contributor

danielhollas commented Nov 21, 2024

Oh no, I am so sorry! This is what happens when one comments on Github late at night. :-)
Thanks for letting us know! 🫶

@yakutovicha
Copy link
Member

related to #54 ?

@edan-bainglass
Copy link
Member

edan-bainglass commented Dec 4, 2024

related to #54 ?

Pinging @giovannipizzi

Thanks for tagging this @yakutovicha. Indeed, this could be used to cover already the demo-server AND first-time-user notifications. Perhaps we should consider integrating this into the very top of ANY app, including the home app. This will could then truly be a general user-notification system. If we do so, we may want to consider if it should be independent of other top-level warnings, such as bug reports.

I think for now maybe it's best we push ahead with something simpler for the upcoming release. But we should indeed look into this sooner than later, to avoid deviations with other developments.

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

Successfully merging a pull request may close this issue.

7 participants