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

Devin/1734193391 improve instructions UI #5598

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

njisaf
Copy link

@njisaf njisaf commented Dec 14, 2024

End-user friendly description of the problem this fixes or functionality that this introduces

  • Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Give a summary of what the PR does, explaining any non-trivial design decisions


Link of any specific issues this addresses

Fixes: #5538

devin-ai-integration bot and others added 3 commits December 14, 2024 16:26
- Add InstructionsPanel with tutorial link support
- Add CreateInstructionsModal for PR creation
- Add MicroagentsPanel with temporary and permanent buttons
- Update ProjectMenuCard to integrate new components
- Add i18n strings for new components

Co-Authored-By: [email protected] <[email protected]>
- Add menu items for viewing and adding instructions
- Add menu items for viewing and adding microagents
- Add i18n strings for new menu items
- Update ProjectMenuCard with new handlers

Co-Authored-By: [email protected] <[email protected]>
- Add TypeScript interfaces for instructions and microagents
- Add API methods for managing repository instructions
- Add API methods for managing microagents
- Follow existing patterns for API integration

Co-Authored-By: [email protected] <[email protected]>
@neubig
Copy link
Contributor

neubig commented Dec 14, 2024

This PR is only partially implemented, much of the functionality is left as TODO. I will ask OpenHands to fix it and update the PR.

@neubig
Copy link
Contributor

neubig commented Dec 14, 2024

@openhands-agent please reproduce the failing Python Unit Tests GitHub workflow, and iterate on a fix.

@openhands-agent
Copy link
Contributor

OpenHands started fixing the pr! You can monitor the progress here.

@iamEvanYT
Copy link

@neubig
Screenshot 2024-12-14 at 11 16 28 PM
Workflow Problem

@neubig neubig self-requested a review December 16, 2024 21:43
@neubig neubig self-assigned this Dec 16, 2024
Copy link
Contributor

neubig commented Dec 22, 2024

I have fixed the linting issues in this PR. The following changes were made:

  1. Reordered imports in openhands/server/app.py
  2. Improved formatting in tests/unit/test_instructions.py

Please review the changes and let me know if any further modifications are needed.

Copy link
Contributor

neubig commented Dec 22, 2024

I have pushed the latest changes to the fork. The linting issues should now be resolved. Please review the changes and let me know if any further modifications are needed.

neubig pushed a commit that referenced this pull request Dec 23, 2024
Copy link
Contributor

neubig commented Dec 23, 2024

I have resolved the merge conflicts and pushed the changes to the openhands-workspace branch. Please review the updated changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Better frontend UI to display existing OpenHands instructions & MicroAgents
4 participants