Skip to content

fix: issues-7.3#310

Merged
AliAlaa88 merged 2 commits into
developmentfrom
fix/issues-7
Dec 15, 2025
Merged

fix: issues-7.3#310
AliAlaa88 merged 2 commits into
developmentfrom
fix/issues-7

Conversation

@bedosaber77

Copy link
Copy Markdown
Collaborator

This pull request introduces several improvements to tweet handling and cache invalidation logic, focusing on ensuring UI consistency after tweet actions and improving data structure clarity. The main changes include updating cache invalidation to refresh the timeline after deleting a reply, enhancing the handling of follower counts, and extending the Content type to include mentions.

Cache Invalidation Improvements:

  • The cache invalidation logic now also refreshes the timeline (/timeline/following) after a reply is deleted, ensuring the UI stays up-to-date when replies are removed.

Follower Count Handling:

  • Refactored the calculation of follower counts for both tweets and parent tweets to improve readability and maintainability. The logic now uses multi-line formatting for clarity. [1] [2]

Mentions Support:

  • The Content interface now includes an optional mentions field, allowing tweets to explicitly list user mentions.
  • The computed parentContent in the tweet component now populates the mentions field from the parent tweet, ensuring mentions are available for downstream features.

Tweet Actions Enhancement:

  • The useTweetActions composable is now called with both the tweet ID and the parent tweet ID, enabling more context-aware tweet operations.## Description

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactoring
  • Performance improvement

Related Issues

Closes #

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code where necessary
  • My changes generate no new warnings or errors
  • I have added unit tests that prove my fix/feature works
  • New and existing unit tests pass locally
  • The Design is screen responsive

Reviewer

@AliAlaa88

@AliAlaa88 AliAlaa88 merged commit b637071 into development Dec 15, 2025
2 of 3 checks passed
@sonarqubecloud

Copy link
Copy Markdown

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.

2 participants