-
Notifications
You must be signed in to change notification settings - Fork 27
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
Cannot delete filters from 'Edit Filters' tab #476
Comments
Okay, I'll look into it. As a workaround have you tried going to the Data Admin tab (top right menu), and then Filters, and then deleting? |
This looks like you didn't compile the JavaScript. Can you explain how you installed MegaQC? |
Hi, I am working with kr0401. The install was done using the docker compose stack using the directions here. I assumed that compiled code was present inside the containers, does something need to be built outside of the containers for them to be fully functional? |
Oh, yes it should be pre-compiled when using Docker. Can you please show the browser logs when you hit the Data Admin page? |
And while we're at it, can I have the frontend logs (ie the browser console) and the backend logs (from Docker Compose) when you hit the existing Delete button? |
Clicking delete button: backend:
|
Looking at the "Edit Sample Filters" page now, I can't see how it ever worked. Maybe there was some JavaScript that got deleted at some point? In any case I think I'll replace it with a link to the appropriate data admin page. |
We ran into some issues trying to test the changes. Will these changes work if we are using the docker-compose install or will we have to use the standard install to see the updates? When trying to recompile the JS we ran into an error while running the second command.
Do you have any guesses as to what we are doing wrong? |
I think that's just an error on Linux that I haven't noticed yet on MacOS, but it's still a bug because I need to update webpack. Anyway, the workaround for now is to |
We have tested fix #479 and recompiled the javascript but it doesn't seem to be working for us. The data admin page is still completely empty though according to the fix we at least should be seeing a filters resource in the admin panel. I have attached a picture of my data admin page as well as the browser console while on the page. Any further ideas on why we may not be seeing the changes? |
It's not "empty" so much as the JavaScript is throwing an error causing the page to not render. This is odd because the dependencies should be locked to exactly the same ones I use while developing. I'll test this on a fresh install and see what happens. |
Any updates on if you were able to reproduce with a fresh install? @multimeric |
The "Delete" button does not work under the "Edit Filters" tab.
The hover over effect does work, the button will appear red but nothing happens when I click it. We run MegaQC through Docker compose with only one minor change to the amount of threads requested. (See #475 for info on what was changed during install)
To Reproduce
Expected behavior
The filter would be removed from the list of filters.
Additional context
In the image below you can see the delete button is highlighted red from the hover over and the lighter band of red outlining it that appears after the button has been clicked. This visual change tells me the site is recognizing the button has been clicked but nothing else happens. This is all being done from the admin account.
The text was updated successfully, but these errors were encountered: