-
Notifications
You must be signed in to change notification settings - Fork 143
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
adding another userful apps #34
Conversation
Just added another frappe apps
corrected githublink
Hi! @linkmk Thanks for the contribution. I will test this addition and merge once completed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you revert the repo name in the commit to the main?
@flexcomng that would actually be a cool addition. Can I just make the correct pull request? |
@fthobe yes please. I will close this pull request now so you can make the correct pull request. |
Have there been any significant changes we need to consider? |
No significant changes but i was considering moving additional app installs to after ssl install in case any app install breaks which is often the case with HRMS, the installation will still be deemed completed. |
@flexcomng done in PR #40 |
I just cloned HRMS code block to install another frappe apps like CRM, helpdesk and insights in this script.