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

Add install plan for metadeploy #28

Merged
merged 4 commits into from
Oct 17, 2019
Merged

Add install plan for metadeploy #28

merged 4 commits into from
Oct 17, 2019

Conversation

davisagli
Copy link
Contributor

This defines the steps that should be run by MetaDeploy to install Outbound Funds. With this, I can publish a new version by running a command like cci task run metadeploy_publish -o tag release/1.15 -- this is something that Jason or I will need to do for now, because it requires admin access to MetaDeploy.

The plan includes a check so that OutboundFundsNPSP will be installed if and only if NPSP is installed.

I've already published a draft version of the installer which can be tested at https://install.salesforce.org/products/outbound-funds -- it's possible to add more explanatory text here if you want.

Critical Changes

Changes

Issues Closed

@davisagli davisagli requested a review from Nickers3 July 11, 2019 18:39
@davisagli davisagli marked this pull request as ready for review July 11, 2019 18:41
@Nickers3
Copy link
Contributor

@davisagli, this looks really slick. One thing to mention is we will have instances in which people would only install the core. It looks as though both are listed as required in the plan. Could the NPSP Ext be listed in the additional plans section vs. a part of the core plan?

@Nickers3
Copy link
Contributor

Also, I tried to use the installer for a developer org and after the pre-install validation completed, I wasn't able to install. Is that by design?

image

@davisagli
Copy link
Contributor Author

@Nickers3 the pre-install validation checks whether NPSP is installed in the org already. If not, the step for the extension package will be skipped. Is that sufficient to handle the case where only the core should be installed, or are there cases where the org has NPSP but the extension package is not desired?

I'll check why it's not working -- sorry for using you as the guinea pig and not actually trying it myself yet.

@Nickers3
Copy link
Contributor

@davisagli, happy to be the guinea pig as this is all really excited to see unfold. Gotcha on the pre-install validation check and the extension package being dynamic based on NPSP.

There could be situations where someone has NPSP and only use the core. Most of the time if they have NPSP, they'd want both. @wcorkill, would you and Shari agree with that statement?

How much effort would it be to allow someone to choose if they want the Extension package? Might be worth a quick drive by Bill and Shari about this idea. I'll post on quip to gauge their thoughts.

@davisagli
Copy link
Contributor Author

@Nickers3 we can make the extension package optional, with a checkbox that is checked by default.

@Nickers3
Copy link
Contributor

Nickers3 commented Jul 11, 2019 via email

@Nickers3
Copy link
Contributor

@davisagli @wcorkill and Shari agree, let's make the extension package optional.

@davisagli
Copy link
Contributor Author

@Nickers3 the extension package is now optional, and I've figured out a workaround for the bug with the install button remaining disabled.

@Nickers3
Copy link
Contributor

Nickers3 commented Jul 12, 2019

@davisagli, one last error I noticed while testing - if I had an install error like the attached, I can't re-rerun the precheck validation
Error trying to upgrade from 1.13 to 1.15 (the master detail to lookup change)
image

Unable to re-run the pre-install validation after I went through the install process.
image

@Nickers3
Copy link
Contributor

If I refresh the page, I can re-run the validation and install. So it is just staying on the page that causes the issue.

@davisagli
Copy link
Contributor Author

@Nickers3 Is it all right if I merge this so the repository includes the code that was used to publish the installer that has been live for a while now?

@Nickers3
Copy link
Contributor

Nickers3 commented Oct 17, 2019 via email

@davisagli davisagli merged commit b7c9a5d into master Oct 17, 2019
@davisagli davisagli deleted the feature/installer branch October 17, 2019 01: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.

2 participants