-
Notifications
You must be signed in to change notification settings - Fork 29
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
Build your own custom dashboard #195
Comments
@armenzg Could you please let me know what this issue is all about? |
Do you remember the two config files you were handling it on the last PR? Imagine a page that the user could add the benchmarks they want to see instead of a preset. The benchmarks chosen would then be stored locally it in their |
@armenzg yes I do, so you mean some if else condition that say load presets from local storage if there aren't any then go with default one, but how they will chose the benchmarks that they want to store, like any specific UI needs to be implemented for that? |
You can use this page as inspiration: You can even copy the code from treeherder into the project if you like. |
Thanks @armenzg I will have a look 👍😅. |
@armenzg here https://treeherder.mozilla.org/perf.html#/graphs we have 2 buttons when to select the date and other is to select the test data, do you also plan to have a dropdown for date as well or only a button so that user can add the benchmarks. |
Just the benchmarks. Thanks! |
@divyanshu-rawat how is it going? Please let me know if you're still going to pursue this or if I should find another contributor. Have a good weekend. |
@armenzg I haven't started working on will work on it in upcoming week, I am occupied these days with my full time job & some personal stuffs. :) , I hope you understand. 👍 |
Of course! Thanks for updating me! |
@armenzg I will start working on it tomorrow , will ping you If I face any problems. 👍 |
Nbp mentions:
The text was updated successfully, but these errors were encountered: