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.

Sign up for GitHub

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: WS switcher and LHN are not reset to Expensify after leaving #45285

Merged
merged 14 commits into from
Jul 31, 2024
Merged

Fix: WS switcher and LHN are not reset to Expensify after leaving #45285

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
bc0168d
Fix: WS switcher and LHN are not reset to Expensify after leaving
truph01 Jul 11, 2024
f306ab2
Fix: Move effect to BottomTabNavigator
truph01 Jul 11, 2024
d554f68
Fix: Only subscribe active workspace ID
truph01 Jul 16, 2024
f490e28
merge main
truph01 Jul 18, 2024
2778283
Fix: Resolve conflict
truph01 Jul 23, 2024
ddd1819
merge main
truph01 Jul 24, 2024
6c2550f
Fix: Move useEffect to BaseSidebarScreen
truph01 Jul 24, 2024
6885046
Fix: Lint
truph01 Jul 24, 2024
9253a2c
Fix: Lint
truph01 Jul 24, 2024
92bb5aa
Fix: Lint
truph01 Jul 24, 2024
08f2deb
rename variable
truph01 Jul 25, 2024
6e76a9d
Feat: Merge latest main
truph01 Jul 30, 2024
baaca67
Feat: Remove didResetActiveWorkspaceIDRef
truph01 Jul 30, 2024
e1c650d
Feat: Move useOnyx before useEffect
truph01 Jul 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix: Lint
  • Loading branch information
@truph01
truph01 committed Jul 24, 2024
commit 9253a2c1bb509952299ece6792da01394e44e70c
2 changes: 1 addition & 1 deletion src/pages/home/sidebar/SidebarScreen/BaseSidebarScreen.tsx
View file
Original file line number Diff line number Diff line change
@@ -37,7 +37,7 @@
const [selectedPolicy] = useOnyx(`${ONYXKEYS.COLLECTION.POLICY}${activeWorkspaceID ?? -1}`);
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
const [selectedPolicy] = useOnyx(`${ONYXKEYS.COLLECTION.POLICY}${activeWorkspaceID ?? -1}`);
const [activeWorkspace] = useOnyx(`${ONYXKEYS.COLLECTION.POLICY}${activeWorkspaceID ?? -1}`);

NAB. Naming consistency


useEffect(() => {
if (selectedPolicy || activeWorkspaceID === undefined || didResetActiveWorkspaceIDRef.current) {
if (selectedPolicy || activeWorkspaceID === undefined || !!didResetActiveWorkspaceIDRef.current) {

Check failure on line 40 in src/pages/home/sidebar/SidebarScreen/BaseSidebarScreen.tsx

GitHub Actions / Run ESLint 

Prefer using nullish coalescing operator (`??`) instead of a logical or (`||`), as it is a safer operator
return;
}

Loading