## Bug description

If I mark an insight as a favorite (⭐) and then refresh, the insight is no longer marked as a favorite although it is (can see it in Favorites).

<img width="390" alt="image" src="https://user-images.githubusercontent.com/30181361/182802236-9ecf0531-899c-4f25-ae86-bc233cefdc2f.png">

Note that I'm in the _Favorites_ tab and that _Bug example_ is not starred.
## Environment

- [x] PostHog Cloud
- [ ] self-hosted PostHog, version/commit: please provide

posthog/api/insight.py
from posthog.settings import SITE_URL
from posthog.tasks.update_cache import synchronously_update_insight_cache
from posthog.utils import DEFAULT_DATE_FROM_DAYS, get_safe_cache, relative_date_parse, should_refresh, str_to_bool
logger = structlog.get_logger(__name__)
def log_insight_activity(
    activity: str,
    insight: Insight,
    insight_id: int,
    insight_short_id: str,
    organization_id: UUIDT,
    team_id: int,
    user: User,
    changes: Optional[List[Change]] = None,
) -> None:
    """
    Insight id and short_id are passed separately as some activities (like delete) alter the Insight instance
    The experiments feature creates insights without a name, this does not log those
    """
    insight_name: Optional[str] = insight.name if insight.name else insight.derived_name
    if insight_name:
        log_activity(
            organization_id=organization_id,
            team_id=team_id,
            user=user,
            item_id=insight_id,
            scope="Insight",
            activity=activity,
            detail=Detail(name=insight_name, changes=changes, short_id=insight_short_id),
        )
class InsightBasicSerializer(TaggedItemSerializerMixin, serializers.ModelSerializer):