Skip to content

Conversation

@skdud4659
Copy link
Member

Skip Review (optional)

  • Minor changes that don't affect the functionality (e.g. style, chore, ci, test, docs)
  • Previously reviewed in feature branch, further review is not mandatory
  • Self-merge allowed for solo developers or urgent changes

Description (optional)

SSIA

Things to Talk About (optional)

Signed-off-by: NaYeong,Kim <nayeongkim@megazone.com>
@skdud4659 skdud4659 requested a review from Copilot May 22, 2025 01:49
@vercel
Copy link

vercel bot commented May 22, 2025

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

3 Skipped Deployments
Name Status Preview Comments Updated (UTC)
console ⬜️ Ignored (Inspect) Visit Preview May 22, 2025 1:49am
dashboard ⬜️ Ignored (Inspect) Visit Preview May 22, 2025 1:49am
web-storybook ⬜️ Ignored (Inspect) Visit Preview May 22, 2025 1:49am

@vercel
Copy link

vercel bot commented May 22, 2025

@skdud4659 is attempting to deploy a commit to the cloudforet Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Contributor

🎉 @yuda110 has been randomly selected as the reviewer! Please review. 🙏

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds a new “Last Synced” column to the ServiceAccountAttachedGeneralAccounts table and provides a formatting slot for its values.

  • Defined the last_synced_at column in the reactive state
  • Implemented a <template> slot to format last_synced_at with timezone-aware dayjs
Comments suppressed due to low confidence (1)

apps/web/src/services/service-account/components/ServiceAccountAttachedGeneralAccounts.vue:115

  • Add or update unit/component tests to cover the new last_synced_at column rendering, including correct date formatting and the fallback '-' for null values.
{ name: 'last_synced_at', label: 'Last Synced', sortable: false },

@skdud4659 skdud4659 changed the title feat: add filed at service account attached general accounts table feat: add column at service account attached general accounts table May 22, 2025
Copy link
Member

@yuda110 yuda110 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good~!!

@skdud4659 skdud4659 merged commit 6997ff3 into cloudforet-io:develop May 22, 2025
7 of 8 checks passed
@skdud4659 skdud4659 deleted the feature/bug branch May 23, 2025 05:26
@github-actions github-actions bot mentioned this pull request May 30, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants