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

Fix editfeatureselector example #3846

Merged
merged 1 commit into from
May 4, 2018
Merged

Conversation

llienher
Copy link
Member

@llienher llienher commented May 4, 2018

Stéphane converted the php-date-formatter lib into a webpack custom supported version we can use find in the src/misc directory. Other changes are minor syntax errors in webpack configs and import of css.

@llienher llienher added this to the 2.3 milestone May 4, 2018
@llienher llienher self-assigned this May 4, 2018
@llienher llienher requested review from sbrunner and gberaudo May 4, 2018 09:28
use: {
loader: 'expose-loader',
options: 'DateFormatter'
}
};
};*/
Copy link
Member Author

Choose a reason for hiding this comment

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

@sbrunner Do you prefer we remove these lines or we keep it what a additional informative comment ?

Copy link
Member

Choose a reason for hiding this comment

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

+1 to remove

@@ -165,7 +165,7 @@ const config = {
olcsRule,
angularRule,
typeaheadRule,
dateFormatterRule,
// dateFormatterRule,
Copy link
Member

Choose a reason for hiding this comment

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

same

'proj4': 'proj4/lib',
'jquery-datetimepicker$': 'jquery-datetimepicker/jquery.datetimepicker.js',
Copy link
Member

Choose a reason for hiding this comment

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

extra $

}
};

export default DateFormatter;
Copy link
Member

Choose a reason for hiding this comment

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

missing endline

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

@llienher llienher force-pushed the fix_edit-feature_example branch 4 times, most recently from f270e0d to d3364b9 Compare May 4, 2018 11:59
@llienher llienher changed the title Fix edit feature example Fix editfeatureselector example May 4, 2018
@llienher llienher merged commit 55cd826 into master May 4, 2018
@llienher llienher deleted the fix_edit-feature_example branch May 4, 2018 13:42
sbrunner pushed a commit that referenced this pull request May 4, 2018
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.

3 participants