-
Notifications
You must be signed in to change notification settings - Fork 174
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
[issue_tracker] Add Batch Mode #9339
base: main
Are you sure you want to change the base?
Conversation
Great start Ayush! Could you please embed here (or else link) the video, or screenshots are fine. For the missing modal - could this be fixed before this work is presented to the larger team at the end of September, to avoid confusion and so Debug mode can be fully useful --: The reason why --: @racostas @regisoc let me know asap if this has already been discussed among you. Thanks Ayush, it's looking solid otherwise. |
Hi @ay-bh, I reviewed the PR and is looking like a good start indeed. |
@SantiagoTG, your inputs here are also very welcome since you have been working a loot with issue tracker. If you have time to take a look to this proposal you opinion is very important. Thanks. |
@racostas @ay-bh to keep you updated, we did not have time to test that together with @SantiagoTG. |
Hello @ay-bh great start indeed! Here are some comments from both me and @SantiagoTG. I tried to group comments:
That was a lot more to write than expected, but I tried to describe each point. |
Hi @regisoc, very good review in facts. I'm sure @ay-bh can address all of this points before the end of the coding period (end of October). Maybe he can present some of the advances he had been making in two weeks from now. (then do a second final presentation in November 5). I will keep you posted with more details after consulting it with him because he have other commitments in the school as well. |
Can we also make sure this has a more descriptive name than "Debug View"? I suggest something like "Batch Mode" since that seems to be what it's doing. |
Thank you all for your detailed feedback and suggestions. I'll start working on implementing these improvements. |
Hi @ay-bh, this two changes are looking good. Thanks for the update ! Great work ! |
Brief summary of changes
"Debug View""Batch Mode" in Issue Tracker moduleThe following features will be implemented in a future PR:
issue_tracker_debug_mode_edit
Testing instructions (if applicable)
Link(s) to related issue(s)
Demo
Demo.mov