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.

Sign up for GitHub

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

Jump to bottom

CI and drupal updates #5

Merged
merged 6 commits into from Oct 25, 2022
Merged

CI and drupal updates #5

merged 6 commits into from Oct 25, 2022

Conversation

niquerio
Copy link
Collaborator

@niquerio niquerio commented Oct 24, 2022 •
edited

This PR:

  • adds a dump of the production mysql from 2022-Oct-24. The admin password has been reset to password.
  • makes the drupal image builder and deploy-er action something that is only triggered manually
  • adds an action for building and deploying the site to netlify. It can only be triggered manually right now. Many other options are commented out in case we want them in the future.
  • refactors the JS that gets the drupal json to use environment variables for auth and host
  • adds docker for 11ty development

This PR resolves DS-126 assuming that the netlify ci works which we won't know until we run it.

@niquerio niquerio changed the title 11ty docker CI and drupal updates Oct 24, 2022
@niquerio niquerio marked this pull request as ready for review October 24, 2022 19:06
respinos
respinos requested changes Oct 24, 2022
View reviewed changes
Copy link
Contributor

@respinos respinos left a comment

Choose a reason for hiding this comment

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

The auth needs to be fixed so the menu can be selected and the front page (https://design-system-glowup.netlify.app/) has content.

11ty/src/_data/drupal.js Outdated Show resolved Hide resolved
respinos
respinos approved these changes Oct 25, 2022
View reviewed changes
Copy link
Contributor

@respinos respinos left a comment

Choose a reason for hiding this comment

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

Ship it!

11ty/.env-example
@@ -0,0 +1,3 @@
DRUPAL_AUTH=admin:actual-password
DRUPAL_URL=https://design-system-cms.kubernetes.lib.umich.edu
DEBUG=false
Copy link
Contributor

Choose a reason for hiding this comment

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

Just noting that DEBUG=false will still cause drupal.js to spew output; it's sort of piggy backing on the 11ty DEBUG=Eleventy* mode, which is a lot. Shameless green!

@niquerio niquerio merged commit 757a34c into main Oct 25, 2022
@bridgetburke bridgetburke deleted the 11ty-docker branch February 6, 2023 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@respinos respinos respinos approved these changes

@bridgetburke bridgetburke Awaiting requested review from bridgetburke

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@niquerio @respinos