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

113/hotfix homepage #124

Merged
merged 22 commits into from
Nov 8, 2024
Merged

113/hotfix homepage #124

merged 22 commits into from
Nov 8, 2024

Conversation

nichgalzin
Copy link
Contributor

@nichgalzin nichgalzin commented Nov 8, 2024

Description

Closes #113
This PR set out to fix the issue where the homepage would sometime load the incorrect content when the user had chosen a project not the current one. In the process it has attempt to unify user data fetching the context updates attempting to make the db the SST for the app and moving away from some of the feature we originally used storage for.

Files changed

  • index.ts - adds await to storage on install call
  • selectNewActiveProject.ts - returns changed values to front end and removes from storage
  • updateUserDetails.ts / LoginPage.tsx / SignupPage.tsx / UserTypes.ts - removes values from userSessions
  • HomePage.tsx -updates function and label name
  • SelectProjectButton.tsx / ProfilePage.tsx- updates context store with returned values
  • src/popup.tsx - adds loading states and sets store based on userSession status
  • store.ts - add login user method
  • src/utils/eventAlarmListener.ts - replaces store values with database call and return changed values to from end.
  • src/utils/setEventAlarm.ts - removes awaits

@nichgalzin nichgalzin marked this pull request as ready for review November 8, 2024 14:22
@camelPhonso camelPhonso requested review from camelPhonso and removed request for camelPhonso November 8, 2024 15:09
@camelPhonso camelPhonso enabled auto-merge November 8, 2024 15:12
@camelPhonso camelPhonso disabled auto-merge November 8, 2024 15:16
@camelPhonso camelPhonso merged commit 599b46d into dev Nov 8, 2024
1 check passed
@camelPhonso camelPhonso deleted the 113/hotfix-homepage branch November 8, 2024 15:16
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.

HomePage current project display isn't tracking the user information
2 participants