-
Notifications
You must be signed in to change notification settings - Fork 37
Assets download fix for running from profiles #32
Conversation
I am a little concerned that this change is a little bit too HMRC specific. now that this code is open source we need to think about whether changes are valuable to all users... hardcoding a specific implementation of assets may regress the usefulness of the codebase as an open source project Is it possible to handle assets in more generic way? Note: This may require some refactoring of the existing codebase... |
@@ -95,6 +97,10 @@ def start_from_binary(self): | |||
parent, _ = os.path.split(unzip_dir) | |||
force_pushdir(parent) | |||
|
|||
if "frontend" in self.service_data and self.service_data["frontend"]: |
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.
could we rename this to be "is_frontend", you can also do "if self.service_data.get("is_frontend", False):"
Valid points for refactoring but as we need to get this fix rolled out we'll make these changes in the next iteration. |
Assets download fix for running from profiles
Assets download fix for running from profiles
No description provided.