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

Make SASS compilation work in dev VM again #730

Merged
merged 1 commit into from
Sep 26, 2016

Conversation

ian-ross
Copy link
Contributor

Proposed changes in this pull request

  • Add SASS_PROCESSOR_ROOT setting to development VM settings.
  • This fixes a problem introduced by removing the spurious STATIC_ROOT setting we used to have in the development settings. The Django package we're using for SASS processing was looking for its output files (to check whether they were out of date) either in SASS_PROCESSOR_ROOT or, if that wasn't set, in STATIC_ROOT. Taking out the STATIC_ROOT setting broke the SASS compilation.

When should this PR be merged

Any time.

Risks

None.

Follow up actions

None.

Copy link
Member

@oliverroick oliverroick left a comment

Choose a reason for hiding this comment

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

Some top-notch coding here.

@oliverroick oliverroick merged commit 25d299c into master Sep 26, 2016
@oliverroick oliverroick deleted the bugfix/dev-sass-rendering branch September 26, 2016 11:16
manoramahp pushed a commit to manoramahp/cadasta-platform that referenced this pull request Sep 27, 2016
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