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

FIX: /create_geometry routes GET and POST #195

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jvela018
Copy link
Collaborator

@jvela018 jvela018 commented Jan 8, 2025

Fix to issue #194

Fixed the route to add both methods. Note that in the future it'd be ideal that functions are split into the most simplistic expressions. This way creating webapps becomes a lot easier/straight forward. It's good practice to use tools like Postman for these tasks. In this case it's quite a simple case but it gets hairy very fast otherwise.

With this fix you'll get:

  • When connecting to the backend
    image

  • When creating a geometry
    image

I didn't change the logger's message because I don't think we need an extra layer of complexity there.

@jvela018
Copy link
Collaborator Author

jvela018 commented Jan 8, 2025

@SMoraisAnsys,

Would you know why the Build and Smoke tests are failing? Any pointers are appreciated.

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

Successfully merging this pull request may close these issues.

1 participant