Skip to content

fix frontend crash on google calendar reminder minutes #10034

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

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

itsababseh
Copy link
Contributor

Summary

  • handle numeric enum options correctly in NodeStringInput

Testing

  • yarn format
  • poetry run format
  • yarn type-check
  • yarn test (fails: Process from config.webServer exited early)

Copy link

netlify bot commented May 24, 2025

Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
🔨 Latest commit fed631a
🔍 Latest deploy log https://app.netlify.com/projects/auto-gpt-docs-dev/deploys/6831fd04dfd28000089cc97c

@github-project-automation github-project-automation bot moved this to 🆕 Needs initial review in AutoGPT development kanban May 24, 2025
@github-actions github-actions bot added platform/frontend AutoGPT Platform - Front end platform/backend AutoGPT Platform - Back end labels May 24, 2025
Copy link

deepsource-io bot commented May 24, 2025

Here's the code health analysis summary for commits a51af36..fed631a. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ Success
❗ 3 occurences introduced
🎯 2 occurences resolved
View Check ↗
DeepSource Python LogoPython✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

Copy link

netlify bot commented May 24, 2025

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit fed631a
🔍 Latest deploy log https://app.netlify.com/projects/auto-gpt-docs/deploys/6831fd044451b20008c66732

@AutoGPT-Agent
Copy link

Thank you for submitting this fix for the frontend crash with Google Calendar reminder minutes. The implementation looks solid - properly handling numeric enum options by converting them to strings for display and parsing them back to numbers when needed.

To get this PR ready for merging, please address these items:

  1. Include the standard PR checklist from our template, as it's required for code changes like this. You can mark sections as not applicable if they don't apply, but the checklist itself should be present.

  2. Update the PR title to follow our conventional commit format more precisely by including a scope, such as: fix(frontend): crash on google calendar reminder minutes

  3. It would be helpful to provide a bit more context in the description about what was specifically causing the crash with Google Calendar reminder minutes.

  4. Regarding the test failure you reported (yarn test failing with 'Process from config.webServer exited early'), please clarify if this is related to your changes or if it's a pre-existing issue with the test environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
codex platform/backend AutoGPT Platform - Back end platform/frontend AutoGPT Platform - Front end size/m
Projects
Status: 🆕 Needs initial review
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants