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

Selecting queries for different mines should be made dynamic #217

Open
Manasa2850 opened this issue Mar 14, 2020 · 3 comments
Open

Selecting queries for different mines should be made dynamic #217

Manasa2850 opened this issue Mar 14, 2020 · 3 comments

Comments

@Manasa2850
Copy link
Contributor

Currently, every time we want to view the tables for a different mine say yeast mine, we need to change the url and query in index.html
Instead of changing that beforehand, it might be better to allow users to enter the url, of the mine they want, in the browser and then reflect the changes accordingly.

@yochannah what do you think of this?

@heralden
Copy link
Member

Although I'm not @yochannah, I will come with my opinion as she hasn't answered yet.

If you are talking about the behaviour when testing im-tables by running npm run dev and opening the dev server in your browser, I think that's a great idea. Adding options to change the results can be useful when developing im-tables, and we can start with being able to change the mine url.

@Manasa2850
Copy link
Contributor Author

@uosl thanks for your response! Yes I'm talking about what we see after running npm run dev

@bhumnay
Copy link

bhumnay commented Mar 18, 2020

I was also thinking about this and maybe we can add a tab somewhere on the very beginning of the main page of the web-app showing the name of current mine ( data is being showed right now)with a drop down options to swap to different mines?
@Manasa2850 @uosl @yochannah

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

3 participants