Skip to content

Conversation

infomiho
Copy link
Collaborator

@infomiho infomiho commented Oct 7, 2025

Description

Based on #531

Fixes #524

This PR reuses the existing dope.sh to create a testable app from an app generated by wasp new app -t saas.

The changes are minimal:

  • change Dummy to SMTP and use the correct email address
  • setup Dotenv Vault

This allows for testing the app in production mode e.g. npx @wasp.sh/wasp-app-runner build.

This PR doesn't setup Stripe, analytics or similar. I think it's not really needed to smoke test the template. We have opensaas.sh with all the bell and whistles for that.

Contributor Checklist

Make sure to do the following steps if they are applicable to your PR:

@infomiho infomiho force-pushed the production-template-testing branch from 744df97 to 3647c22 Compare October 7, 2025 11:47
@infomiho infomiho mentioned this pull request Oct 7, 2025
4 tasks
@infomiho infomiho changed the base branch from main to extract-dope-sh-to-top-level October 7, 2025 11:48
@infomiho infomiho force-pushed the extract-dope-sh-to-top-level branch 2 times, most recently from 8c03552 to 1159ea4 Compare October 7, 2025 11:54
@infomiho infomiho force-pushed the production-template-testing branch from 3647c22 to 8aaf842 Compare October 7, 2025 11:55
@infomiho infomiho marked this pull request as ready for review October 7, 2025 12:10
@infomiho infomiho requested a review from sodic October 7, 2025 12:28
@cprecioso
Copy link
Member

@infomiho haven't looked at anything here, but wanted to pop in; if we're going to be using dope.sh more extensively, might it make sense to migrate it to a more understandable (and less edge-casey) language? We can take advantage of the fact that node is already installed...

@infomiho infomiho force-pushed the extract-dope-sh-to-top-level branch from 1159ea4 to 01c159c Compare October 7, 2025 15:16
@infomiho infomiho force-pushed the production-template-testing branch from 6c8d6e7 to 6a51600 Compare October 7, 2025 15:17
@infomiho
Copy link
Collaborator Author

infomiho commented Oct 7, 2025

@cprecioso I think that the natural journey of all of our scripts -> inline -> bash -> Node.js

I wouldn't do it in this PR though, I'd create an issue if we agree that we want to do it.

@cprecioso
Copy link
Member

@infomiho My vote is to even do that before this PR here. As you see fit.

@Martinsos
Copy link
Member

+1 for the idea of rewrite at some point but also big +1 for not doing it now, priorities again, dope.sh works fine and is tested through use, no need to shake that tree at the moment.

@infomiho infomiho force-pushed the production-template-testing branch from 6a51600 to eb64c15 Compare October 10, 2025 10:32
@infomiho infomiho force-pushed the production-template-testing branch from eb64c15 to 0363f98 Compare October 10, 2025 15:17
@infomiho infomiho changed the base branch from extract-dope-sh-to-top-level to main October 10, 2025 15:18
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.

Simplify testing the Open Saas template in build mode

3 participants