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

fix: Fixed rendering and navigation of all tests in DashboardView and added navigation to a test from sidedrawer also #604

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

sahitya-chandra
Copy link
Contributor

@sahitya-chandra sahitya-chandra commented Feb 20, 2025

Previously, after test creation, the DashboardView remained empty and no tests have been rendered on it.
Also added an onclick navigation to the route of that specific test ManagerView.

Added rendering of all test in navigation header in Drawer.vue.
Navigating to different tests directly from navigation header.

Before:

Screencast.from.2025-02-20.20-23-29.webm
Screencast.from.2025-02-22.19-16-13.webm

After all fixes:

Screencast.from.2025-02-20.20-17-18.webm
Screencast.from.2025-02-22.19-09-57.webm

@saltykheera
Copy link
Contributor

https://ruxailab-prod.firebaseapp.com/
check this @sahitya-chandra
test are getting rendered in manager dashboard may be it is an firebase emulator issue

@sahitya-chandra
Copy link
Contributor Author

sahitya-chandra commented Feb 20, 2025

@saltykheera I am not using firebase emulator. I am using firebase directly. And It is an issue in develop branch.

In production it is working perfectly but not in the code of this repo.

@saltykheera
Copy link
Contributor

saltykheera commented Feb 20, 2025

@saltykheera I am not using firebase emulator. I am using firebase directly. And It is an issue in develop branch.

on which branch the main website is on ?
did you tried on main website
because in my local host it is working

@sahitya-chandra
Copy link
Contributor Author

sahitya-chandra commented Feb 20, 2025

Production code is very old... I don't know which branch

These part of code is causing issue

In store/modules/test.js ( Wrong fetching of test from myTests field because it is an empty object in DB)

Screenshot from 2025-02-20 21-49-00

DashboardView.vue (testDocId does not even exist in tests collection)

Screenshot from 2025-02-20 21-53-37

@saltykheera
Copy link
Contributor

@marcgc21 on what branch the current production is working ?

Copy link

@sahitya-chandra sahitya-chandra changed the title fix: Fixed rendering of tests in DashboardView and fixed navigation to that specific test ManagerView fix: Fixed rendering of all tests in DashboardView and navigation to that test's managerview and navigation from Drawer also Feb 22, 2025
@sahitya-chandra sahitya-chandra changed the title fix: Fixed rendering of all tests in DashboardView and navigation to that test's managerview and navigation from Drawer also fix: Fixed rendering and navigation of all tests in DashboardView and navigate to a test from navigation drawer also Feb 22, 2025
@sahitya-chandra
Copy link
Contributor Author

@marcgc21 @jvJUCA please review this code...

@sahitya-chandra sahitya-chandra changed the title fix: Fixed rendering and navigation of all tests in DashboardView and navigate to a test from navigation drawer also fix: Fixed rendering and navigation of all tests in DashboardView and added navigation to a test from sidedrawer also Feb 22, 2025
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

Successfully merging this pull request may close these issues.

2 participants