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: support export async function #61

Merged
merged 1 commit into from
Mar 1, 2023
Merged

fix: support export async function #61

merged 1 commit into from
Mar 1, 2023

Conversation

egreb
Copy link
Contributor

@egreb egreb commented Mar 1, 2023

Hey, really like this plugin.

We noticed that actions and loaders declared as async functions are not being correctly exported by the routes generator.

Looked into the code and seems like it was because the regex is not matching for async function.

Not sure if there is a reason for this, but created a small PR to support it.

Let me know if there is anything else that needs to be done to solve this.

Copy link
Owner

@oedotme oedotme left a comment

Choose a reason for hiding this comment

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

Hey @egreb, I believe that was only uncovered case for defining loader/action function.

Looks good, thanks for adding that!

@oedotme oedotme merged commit c922b59 into oedotme:main Mar 1, 2023
@oedotme
Copy link
Owner

oedotme commented Mar 3, 2023

Published at v1.7.20

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.

2 participants