Feat/tutorial#355
Merged
Merged
Conversation
…ourButton components
- Added tour context to ConfigureToolModal, RightBar, ConverterList, and FormConverterSection to manage tour steps. - Implemented tour attributes for buttons and tabs in RightBar and ConverterList. - Enhanced ParameterStepConverter and ScopeStepConverter to trigger tour steps upon actions. - Updated Dataset and Explorer components to include tour steps and attributes for better navigation. - Adjusted tour steps in datasetsTour.js and notebookTour.js to reflect new UI changes and guide users effectively. - Improved layout and scrolling behavior in FormExplorerSection and other components for better usability during tours.
Remove console.log for debug.
… and Save Dataset modal
…r improved readability and consistency. - Updated homeTour.js, notebookTour.js, and index.js to standardize string quotes and formatting. - Added experimentsTour steps and configurations to index.js. - Enhanced notebookTour.js with clearer descriptions and consistent formatting. - Integrated tour context in Results.jsx and ResultsDialogLayout.jsx for better user guidance. - Added data-tour attributes to various components for improved tour navigation. - Cleaned up unused code and ensured consistent styling across components.
Member
|
There is a big conflict in Datasets.jsx view. For some reason we cant just keep develop changes |
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request introduces an interactive guided tour system using react-joyride to help users navigate through different modules of the DashAI application. The tour provides step-by-step guidance for key workflows including home navigation, dataset upload, notebook creation, and experiment configuration.
Key Changes:
- Added react-joyride dependency for interactive tours
- Implemented tour infrastructure with TourProvider, TourButton, and useTour hook
- Created tour configurations for Home, Datasets, Notebook, and Experiments modules
- Added data-tour attributes throughout components for tour targeting
- Removed debug console.log statements from production code
Reviewed Changes
Copilot reviewed 61 out of 63 changed files in this pull request and generated 13 comments.
Show a summary per file
| File | Description |
|---|---|
| DashAI/front/yarn.lock | Updated dependencies including react-joyride and various Babel/build packages |
| DashAI/front/src/hooks/useTour.js | New hook managing tour state and localStorage persistence |
| DashAI/front/src/components/tour/* | New tour components (TourProvider, TourButton) and styling |
| DashAI/front/src/constants/tours/* | Tour step definitions and configurations for each module |
| DashAI/front/src/pages/* | Integration of TourProvider and data-tour attributes |
| DashAI/front/src/components/notebooks/* | Added tour integration points and data-tour attributes |
| DashAI/front/src/utils/jobPoller.js | Removed debug console.log statements |
| tests/back/job_queue/test_huey_job_queue.py | Added time.sleep() calls for test stability |
Comments suppressed due to low confidence (1)
DashAI/front/src/constants/tours/experimentsTour.js:1
- Unused import style.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
cristian-tamblay
approved these changes
Nov 5, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces guided tour functionality throughout the experiment creation and management workflow in the DashAI frontend, and improves backend dataset creation error handling. The main changes include adding context-aware tour step advancement, data attributes for tour step identification, and enhanced duplicate dataset error handling.
Guided Tour Integration:
useTourContextusage and logic to advance the tour step after key user actions (e.g., opening dialogs, adding models, progressing experiment steps) inConfigureModelsStep.jsx,ExperimentsTable.jsx,NewExperimentModal.jsx, andRunnerDialog.jsx. [1] [2] [3] [4]data-tourattributes to critical UI elements (e.g., model selector, add model button, experiments table, runner button, autocomplete fields) to enable precise tour step targeting. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]Frontend Feature Enhancements:
react-joyridedependency topackage.jsonto support the guided tour functionality.Backend Improvements:
datasets.pyby checking for existing dataset names before creation and raising a 409 Conflict error if a duplicate is found, both proactively and on database integrity errors. [1] [2]Minor Frontend Code Quality Updates:
ItemSelectorWithInfo.jsxfor better component flexibility.RunnerDialog.jsx. [1] [2]These changes collectively enhance user onboarding with a guided tour, improve error handling, and increase UI flexibility.