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

feat(trending): implement trending feature and components #820

Merged
merged 8 commits into from
Oct 9, 2024
Merged

Conversation

Innei
Copy link
Member

@Innei Innei commented Oct 9, 2024

CleanShot 2024-10-09 at 1  03 45@2x

Copy link

vercel bot commented Oct 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
follow ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 9, 2024 5:07am

@Innei
Copy link
Member Author

Innei commented Oct 9, 2024

/ai-review

@follow-pr-reviewer-bot
Copy link

Suggested PR Title:

feat(trending): implement trending feature and components

Change Summary:
Added trending feature with fetching and display components, new icons, and enhanced modal interactions. Updated localization strings and package dependencies for new feature support.

Code Review:
No change requests necessary.

Copy link

github-actions bot commented Oct 9, 2024

Linting and formatting issues were automatically fixed. Please review the changes.

@Innei Innei changed the title feat: trending modal feat(trending): implement trending feature and components Oct 9, 2024
@Innei Innei merged commit 6c4c543 into dev Oct 9, 2024
3 checks passed
@Innei Innei deleted the feat/trending branch October 9, 2024 09:58
Copy link

sentry-io bot commented Oct 21, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ TypeError: Cannot read properties of undefined (reading 'map') /Applications/Follow.app/Contents/Resources/app... View Issue
  • ‼️ TypeError: Cannot read properties of undefined (reading 'slice') /Applications/Follow.app/Contents/Resources/app... View Issue

Did you find this useful? React with a 👍 or 👎

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.

1 participant