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

refactor(content-sidebar): convert AnnotationActivityMenu to TypeScript #3893

Merged

Conversation

devin-ai-integration[bot]
Copy link
Contributor

Description

Convert AnnotationActivityMenu component from Flow to TypeScript.

Changes

  • Converted AnnotationActivityMenu.js to TypeScript (.tsx)
  • Preserved original file as AnnotationActivityMenu.js.flow for Flow compatibility
  • Updated ButtonType import to use correct module
  • All tests and lint checks pass

Testing

  • Ran yarn tsc - no errors
  • Ran yarn lint:js and yarn lint:ts - no errors
  • Ran yarn test - all tests pass

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

Copy link
Contributor Author

🤖 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

@devin-ai-integration devin-ai-integration bot force-pushed the devin/1738726985-convert-ts-annotation-activity-menu branch from af47f1c to 7997517 Compare February 5, 2025 18:26
@devin-ai-integration devin-ai-integration bot marked this pull request as ready for review February 5, 2025 18:26
@devin-ai-integration devin-ai-integration bot requested a review from a team as a code owner February 5, 2025 18:26
@devin-ai-integration devin-ai-integration bot changed the title chore(js-ts): Convert AnnotationActivityMenu to TypeScript refactor(content-sidebar): convert AnnotationActivityMenu to TypeScript Feb 5, 2025
Copy link
Contributor

mergify bot commented Feb 6, 2025

This pull request has been removed from the queue for the following reason: checks failed.

The merge conditions cannot be satisfied due to failing checks

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.

If you want to requeue this pull request, you need to post a comment with the text: @mergifyio requeue

@mergify mergify bot merged commit 50d6c3f into master Feb 6, 2025
8 checks passed
@mergify mergify bot deleted the devin/1738726985-convert-ts-annotation-activity-menu branch February 6, 2025 06:16
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