-
-
Notifications
You must be signed in to change notification settings - Fork 817
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
Package Riverlea theme extension during CiviCRM build #31324
base: master
Are you sure you want to change the base?
Conversation
🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷 Introduction for new contributors...
Quick links for reviewers...
|
Hmmm, now we've got a more different error... so I guess that's progress... I'm gonna try the job again cause maybe it's extension directory timeout. |
um, riverlea isn't in the extension directory tho… I guess naîvely I thought it might pick it up from lab.civicrm.org/extensions |
Test this please. (Have added to the extensions directory). |
Test this please |
@ufundo run-distmaker is failing |
Looks like the fail was downloading iatspayments info from the extension feed. The url in the logs seems valid and works for me, if slowly - so my guess is its just a network timeout issue during the job. I've hit retry and crossed my fingers |
Seems to pass… test this please |
It's not there… It is now on the AddNew screen - but I think that's just via the updated extdir listing. |
Resets checkbox width for text-rendered checkbox output on confirmation screens
Overview
Following discussion this packages the most recent release of the RiverLea theme framework extension (https://lab.civicrm.org/extensions/riverlea) during the CiviCRM build process.
Before
RiverLea isn't packaged, so needs to be installed manually.
After
It is. Anyone can enable RiverLea to try one of its four subthemes/'streams'.
Technical Details
RiverLea is a large extension. To better understand it, a good starting point is to look at the GitLab readme, changelog, issues, & PR history.
Comments
I am kind of guessing this is the right way to add this, but cannot test.