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(core): add update name and avatar endpoint #6636

Merged
merged 1 commit into from
Sep 29, 2024

Conversation

wangsijie
Copy link
Contributor

Summary

Implement the API PATCH /profile to update name and avatar, will support more fields later.

Testing

Integration tests.

Checklist

  • .changeset
  • unit tests
  • integration tests
  • necessary TSDoc comments

@github-actions github-actions bot added the feature Cool stuff label Sep 25, 2024
Copy link

COMPARE TO master

Total Size Diff 📈 +3.77 KB

Diff by File
Name Diff
packages/core/src/routes/profile/index.openapi.json 📈 +914 Bytes
packages/core/src/routes/profile/index.ts 📈 +1.05 KB
packages/integration-tests/src/api/api.ts 📈 +90 Bytes
packages/integration-tests/src/api/profile.ts 📈 +337 Bytes
packages/integration-tests/src/api/verification-record.ts 📈 +4 Bytes
packages/integration-tests/src/helpers/admin-tenant.ts 📈 +303 Bytes
packages/integration-tests/src/helpers/profile.ts 📈 +522 Bytes
packages/integration-tests/src/tests/api/profile/index.test.ts 📈 +1.19 KB

Copy link
Contributor

@simeng-li simeng-li left a comment

Choose a reason for hiding this comment

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

overall LGTM

@wangsijie wangsijie merged commit 3131802 into master Sep 29, 2024
36 checks passed
@wangsijie wangsijie deleted the wangsijie-log-9889-edit-name-and-avatar branch September 29, 2024 02:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants