[Bug] Refactor this function to not always return the same value.

[telebovich]

Bug Description

The intention of this function is not clear. Take a look.

  const shouldShowForCurrentView = useCallback(() => {
    if (!currentView) return true;
    
    // Show for all views
    return true;
  }, [currentView]);

Did you mean false in the if statement?

Steps to Reproduce

Take a look at the function and tell if there is any logic in it. What was the original intent?

Expected Behavior

Should return different values or this is not needed at all.

Browser and Version

Edge latest

AppFlowy Version(s)

0.15

Screenshots

No response

Logs and Console Output

No response

Additional Context

No response