-
Notifications
You must be signed in to change notification settings - Fork 10
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
Testing op-admin-dashboard #21
Comments
Testing Home TabInformation displayed: Boxes
Plot
|
@AlirezaRa94 FYI, in case you want to get a head start on fixing these issues |
this error indicates that the mongodb container was not actually up, or you were unable to connect to it.
Can you verify that the container was running at the time you ran that test? |
I guess other errors are related to the database connection as well. Please let me know if this is not the case. |
@AlirezaRa94 it looks like the trip table was working, so not sure this is the problem on all tabs |
It seems that at the beginning, the connection was proper, but after that, it lost its connection. When you filter data using those date filters, it tries to get data from the database again, and I think the first error is because of the lost connection. |
@swastis10 I was able to run the dashboard yesterday without running into the errors that you did, but I only loaded one program. Have you been able to retry with checking whether the database was running correctly? |
@shankari @AlirezaRa94 all the above mentioned issues got resolved once I restarted the mongodb container |
Testing Push notification tabSome use cases of this Push notification tab are :
|
@swastis10 @AlirezaRa94
Once done, you should see the admin dashboard, not the public dashboard. |
@shankari In cognito, when I go to our NREL user pool, I see public dashboard URL as the hosted UI (See attached screenshot). We either need to change the callback to localhost link or need to host admin dashboard on Stage so that we can view that UI once we sign in through cognito. |
@AlirezaRa94 how did you configure this for your testing? |
@shankari @swastis10 |
After setting callback URL to http://localhost:8050, I was able to load admin dashboard after Cognito login. There was another error I encountered: Logs: Another error: |
@swastis10 |
@AlirezaRa94 it looks like I can't regenerate new tokens? I then tried to regenerate with program = Screen.Recording.2023-03-22.at.9.01.41.AM.movNote that when I export QR codes, I do see them, but they are not visible in the UI.
There is also no method to remove tokens (in case the admin wanted to change the format, etc). |
@AlirezaRa94 I think it is correct. Here are is my config.py, let me know if I need to make some other changes: In Cognito - Allowed callback URLs is set to http://localhost:8050 |
@shankari I couldn't find the method to remove tokens from the database (emission/storage/decorations/token_queries.py). Could you point me to this method? |
@AlirezaRa94 I see the pagination now, thanks. I can think of two possible approaches:
@AlirezaRa94 which is easier for you to implement? I have a mild preference for (1) because it mirrors the workflow for which we designed this, and seems to be easier to implement We can always tweak later based on admin feedback. |
@shankari Another point to mention about the token page is that it displays all the tokens available in the database, including the new ones. It also exports all of them, which I believe is not the intended behavior. |
@AlirezaRa94 I think it would be good to show both but separate them |
@AlirezaRa94 production dockerfile does not work; container continually fails start with
That's why I ask you to indicate the testing done: |
@shankari |
@AlirezaRa94 , I have created a test App client on NREL AWS console with the following settings: In Config.py, my configs are as follows:
|
@swastis10 are there any error logs? For that matter, where do you see logs in a dash app? |
@swastis10 I can sign in without any problems with the config that you have provided. Can you check the config.py file inside the docker container and see if it is the same? |
Closing this for now since portmanteau issues are hard to debug and resolve |
I have started testing op-admin-dashboard. I checked out
dev
branch found here.I loaded data in the dashboard using the script found here. The tar files loaded were -
The dashboard has started to look something like the screenshot attached below.
The text was updated successfully, but these errors were encountered: