Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: Fix import #494

Merged
merged 1 commit into from
Sep 10, 2024
Merged

fix: Fix import #494

merged 1 commit into from
Sep 10, 2024

Conversation

whiterabbit1983
Copy link
Contributor

@whiterabbit1983 whiterabbit1983 commented Sep 10, 2024

🚀 This description was created by Ellipsis for commit e241f7f

Summary:

Added protocol import in agents-api/agents_api/web.py for constructing server URL in scalar_html function.

Key points:

  • Added protocol import from .env in agents-api/agents_api/web.py.
  • protocol is used in scalar_html function to construct server URL.

Generated with ❤️ by ellipsis.dev

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

❌ Changes requested. Reviewed everything up to e241f7f in 10 seconds

More details
  • Looked at 14 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_1cvwt10ppal8Dfp0


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@creatorrr creatorrr merged commit d5f088e into dev Sep 10, 2024
5 of 8 checks passed
@creatorrr creatorrr deleted the x/fix-protocol-variable branch September 10, 2024 16:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants