Skip to content

Conversation

@xkaper001
Copy link
Contributor

Closes #75

📝 Description

Fixes all the bugs mentioned in #75

  • Two splash screens reference in the app (causes breaking).
  • main.dart has two refernces of splash screen which fails the build.
  • the pubspec.yaml file doesn't contain flutter_dotenv but the app uses this package.
  • intl package dependency issues.
  • multiple session folders in lib/presentation
  • multiple widgets folders in lib/presentation
  • splash_screen.dart missed a }, failing the build

🔧 Changes Made

Import Reorganization:

  • Consolidated and cleaned up import statements in therapist/lib/main.dart to improve readability and maintainability.
  • Adjusted import paths for widget files in therapist/lib/presentation/auth/auth_screen.dart, therapist/lib/presentation/home/home_screen.dart, therapist/lib/presentation/profile/profile_screen.dart, and therapist/lib/presentation/session/session_screen.dart to use the widgets subdirectory. [1] [2] [3] [4]

File Deletions:

  • Removed the therapist/lib/presentation/sessions/sessions_screen.dart file as it was no longer needed.
  • Deleted the therapist/lib/presentation/widget/splash_screen.dart file and its associated code.

Dependency Updates:

  • Updated the intl dependency version from ^0.17.0 to ^0.19.0 in therapist/pubspec.yaml.

✅ Checklist

  • I have read the contributing guidelines.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added necessary documentation (if applicable).
  • Any dependent changes have been merged and published in downstream modules.

@xkaper001 xkaper001 changed the title fixed bugs fixed app startup bugs fixes#75 Mar 22, 2025
Copy link
Member

@mdmohsin7 mdmohsin7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mdmohsin7 mdmohsin7 merged commit fe5f478 into AOSSIE-Org:main Mar 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.

BUG: Fix Therapist app bugs on startup

2 participants