liam icon indicating copy to clipboard operation
liam copied to clipboard

♻️ Refactor AppBar: Convert to Server Components and remove use client directives

Open devin-ai-integration[bot] opened this issue 9 months ago • 6 comments

What changes does this PR introduce?

Refactoring:

  • Remove unnecessary 'use client' directives from Avatar/UserAvatarIcon
  • Simplify AppBar architecture by removing wrapper components and handling data fetching directly
  • Define custom getProject function in AppBar component

PR Checklist

  • [x] Added tests related to the changes
  • [x] Updated documentation (if necessary)
  • [x] Added Changeset for incompatible changes
  • [x] Verified that CI passes

Screenshots

None

Additional Information

Refactoring based on PR #1361:

  • https://github.com/liam-hq/liam/pull/1361#discussion_r2048129526
  • https://github.com/liam-hq/liam/pull/1361#discussion_r2048131269
  • https://github.com/liam-hq/liam/pull/1361#discussion_r2048131354

Link to Devin run: https://app.devin.ai/sessions/b9ad63baf00b444eb57ebad3c4e6244d Requested by: [email protected]

⚠️ No Changeset found

Latest commit: 45ad665920366dfa28fd47db2416c1d63492b1d2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Apr 17 '25 04:04 changeset-bot[bot]

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

Name Status Preview Comments Updated (UTC)
liam-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 17, 2025 7:13am
liam-erd-sample ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 17, 2025 7:13am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
liam-docs ⬜️ Ignored (Inspect) Visit Preview Apr 17, 2025 7:13am

vercel[bot] avatar Apr 17 '25 04:04 vercel[bot]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • [ ] Disable automatic comment and CI monitoring

Updates to Preview Branch (devin/1744864520-appbar-refactoring) ↗︎

Deployments Status Updated
Database Thu, 17 Apr 2025 07:10:02 UTC
Services Thu, 17 Apr 2025 07:10:02 UTC
APIs Thu, 17 Apr 2025 07:10:02 UTC

Tasks are run on every commit but only new migration files are pushed. Close and reopen this PR if you want to apply changes from existing seed or migration files.

Tasks Status Updated
Configurations Thu, 17 Apr 2025 07:10:02 UTC
Migrations Thu, 17 Apr 2025 07:10:02 UTC
Seeding Thu, 17 Apr 2025 07:10:02 UTC
Edge Functions Thu, 17 Apr 2025 07:10:02 UTC

View logs for this Workflow Run ↗︎. Learn more about Supabase for Git ↗︎.

supabase[bot] avatar Apr 17 '25 04:04 supabase[bot]

Please write description in English

MH4GF avatar Apr 17 '25 05:04 MH4GF

Please convert to draft

MH4GF avatar Apr 17 '25 05:04 MH4GF