-
Notifications
You must be signed in to change notification settings - Fork 70
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
Onboarding tour for new users #1280
Open
IgnisDa
wants to merge
148
commits into
main
Choose a base branch
from
issue-1259
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains 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
- Remove automatic redirection for authenticated users - Delete unused imports related to authentication - Simplify auth route loader function
- Upgrade various frontend and library dependencies - Update React Router, Vite, esbuild, and other packages - Bump versions of packages like nanoid, isbot, tailwind-merge, and sonner - Synchronize package versions across frontend and website applications
- Create new button component for initiating onboarding tour - Placeholder implementation with console log for tour start
- Create new column to track user's completed onboarding tours - Set column as a text array with a default empty array - Integrate new migration for tracking onboarding tour progress
- Reorganize struct field order for better readability - No functional changes to the User model structure - Maintain existing field types and attributes
- Introduce UserOnboardingTour enum with Media variant - Add completedOnboardingTours field to User model - Update GraphQL types and generated files to support onboarding tour tracking
- Remove UserOnboardingTour enum from enum models - Delete completedOnboardingTours field from User model - Remove related GraphQL and generated type references - Clean up migration and database-related code for onboarding tours
- Replace hardcoded index name with dynamically referenced constant - Improve code maintainability by using predefined index name - Ensure consistent index naming across migration and other components
… enum - Remove `either` crate from multiple Cargo.toml files - Add `ExpireCacheKeyInput` enum in `dependent-models` to replace `Either` usage - Update cache service and related functions to use new enum variant - Simplify cache key expiration logic with more type-specific input
- Extend `ExpireCacheKeyInput` enum with `BySanitizedKey` variant - Implement sanitized key expiration logic in cache service - Support optional user ID-based key sanitization - Enhance cache key expiration flexibility
- Modify `calculate_user_activities_and_summary` function to accept `SupportingService` instead of direct database connection - Update function calls in miscellaneous and statistics services to match new signature - Improve dependency injection and service interaction pattern
… expiration - Add `EnumDiscriminants` derive macro to `ApplicationCacheKey` - Update `ExpireCacheKeyInput` to use `ApplicationCacheKeyDiscriminants` - Modify cache key expiration logic to support discriminant-based sanitized keys - Add cache key expiration for user analytics in user activities calculation
- Update `onboardingTourAtom` type to include `isCompleted` flag - Modify tour completion logic to use atom state instead of localStorage - Add `canTourBeStarted` check to control tour initialization - Refactor tour restart mechanism in profile settings page
- Rename onboarding tour step targets to be more descriptive - Update tour step references across multiple components - Enhance readability of tour step identifiers
- Introduce new onboarding tour step for refresh button - Add tour step targeting the refresh button in movies list page - Update `DisplayListDetailsAndRefresh` component to support tour interaction
- Rename `isTourStarted` to `isTourInProgress` for clarity - Update tour progress check to consider `isCompleted` flag - Modify references across dashboard and media routes - Enhance tour state management with more precise progress tracking
- Update tour start condition to check for undefined tour state - Remove redundant `isTourInProgress` check - Streamline tour initialization in mobile and desktop contexts
- Introduce new onboarding tour step for adding exercises in a workout - Add `AddNewExercise` target to `OnboardingTourStepTargets` enum - Update fitness dashboard route to support new tour step - Include tour step guidance for adding an exercise to an empty workout
- Rename `AddNewExercise` tour step target to `ClickOnAddAnExerciseButton` - Update tour step target in onboarding configuration - Modify button class to match new tour step identifier
- Add `data-exercise-id` attribute to exercise item display component - Enable precise targeting for onboarding tour steps - Improve tour interaction with exercise list items
- Introduce new tour step for selecting an exercise in workout creation - Add `SelectExercise` target to `OnboardingTourStepTargets` enum - Implement tour step advancement when specific exercise is selected - Add conditional targeting for onboarding tour exercise selection
- Add constant `TOUR_EXERCISE_TARGET_ID` for consistent exercise targeting - Update tour step content for exercise selection and addition - Implement tour step advancement when adding selected exercises - Refactor exercise selection targeting using shared constant
- Remove unnecessary fragment navigation - Eliminate unused exercise index tracking - Streamline navigation to fitness action route
- Reorder import statements in fitness route components - Improve import organization for better readability - Remove unnecessary import order variations
- Remove unnecessary backticks artifact in the exercises list component - Clean up extraneous code that does not serve a purpose
- Modify onboarding tour step content to clarify the action of adding exercises to the active workout - Ensure consistency in messaging for user guidance during the onboarding process
…ency - Adjust the order of props in StatInput components to follow length-based organization - Enhance code readability and maintainability by standardizing prop arrangement
- Add new onboarding tour steps for associating weight and reps to exercises - Update `OnboardingTourStepTargets` enum to include `AddWeightToExercise` and `AddRepsToExercise` - Implement conditional class names in `StatInput` for tour step advancement based on user input
… sets - Introduce new onboarding tour step for confirming exercise sets - Update `OnboardingTourStepTargets` enum to include `ConfirmSetForExercise` - Implement conditional class name for tour step advancement in `SetDisplay` component - Enhance user guidance during the exercise confirmation process
- Introduce new onboarding tour step for opening set menu details - Update `OnboardingTourStepTargets` enum to include `OpenSetMenuDetails` - Implement conditional class names in `SetDisplay` for tour step advancement - Enhance user guidance during the set detail adjustment process
…mponent - Simplify class name assignment using clsx for conditional rendering in SetDisplay - Enhance readability by reducing nested ternary operators for onboarding tour steps - Maintain consistency in class name handling for improved code clarity
- Streamline the umami tracking interface by defining the track method directly in the global Window interface - Enhance code clarity by reducing unnecessary type imports
- Remove "@bitprojects/umami-logger-typescript" from devDependencies in package.json - Clean up yarn.lock by removing references to the umami logger and other unused packages - Enhance dependency management by streamlining the lock file
…tails - Introduce new onboarding tour step for opening exercise menu details - Update `OnboardingTourStepTargets` enum to include `OpenExerciseMenuDetails` - Implement conditional logic in `ExerciseDisplay` for tour step advancement - Enhance user guidance during the exercise detail adjustment process
…splay - Remove redundant declaration of `isTourStep` in `ExerciseDisplay` - Introduce a single declaration for `isTourStep` to enhance clarity - Maintain functionality for onboarding tour step advancement during exercise selection
- Introduce new onboarding tour step for finishing the workout - Update `OnboardingTourStepTargets` enum to include `FinishWorkout` - Enhance user guidance with a message upon completing the workout tour - Implement conditional class name logic for tour step advancement in the dashboard
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.
No description provided.