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

Astro Integration #14

Merged
merged 14 commits into from
Nov 5, 2024
Merged

Astro Integration #14

merged 14 commits into from
Nov 5, 2024

Conversation

fahreddinozcan
Copy link
Collaborator

This PR adds Astro Integration to Upstash Workflow, enabling it to be used in Astro Endpoints.

Original work by @hckhanh, the original PR #11 was closed by mistake.

hckhanh and others added 13 commits October 31, 2024 10:15
This change simplifies the function name for better readability and consistency. The adjustment aligns with naming conventions used across the codebase.
Moved Astro from dependencies to devDependencies for better clarity and management. This change keeps runtime dependencies cleaner and ensures Astro is used only in development environments.
Includes essential files such as `extensions.json`, `launch.json`, components, and layouts. Configures basic project structure, TypeScript settings, and environment variables with README documentation.
Co-authored-by: Fahreddin Özcan <88107904+fahreddinozcan@users.noreply.github.com>
Co-authored-by: Fahreddin Özcan <88107904+fahreddinozcan@users.noreply.github.com>
Co-authored-by: Fahreddin Özcan <88107904+fahreddinozcan@users.noreply.github.com>
Reordered function parameters in routeFunction call to match new signature and added missing semicolons for code consistency. These changes ensure smoother API context handling and improve code readability.
@fahreddinozcan
Copy link
Collaborator Author

@hckhanh the previous branch was mistakenly broken by a force push, carried the PR over here. And this looks almost final now. Thanks for the contribution!

@hckhanh
Copy link
Contributor

hckhanh commented Nov 4, 2024

That's great. I hope it's going to be merged soon

@CahidArda CahidArda merged commit 6f58248 into main Nov 5, 2024
1 check passed
@CahidArda CahidArda deleted the astro-integration branch November 5, 2024 07:46
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.

3 participants