-
Notifications
You must be signed in to change notification settings - Fork 88
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
Add gmf.EditFeature service with example #1459
Conversation
@sbrunner Would you please look at the "Same origin" issue I'm having ? Thanks. |
61b3708
to
d98dad0
Compare
@fredj You can review right away, if you want. |
da88467
to
abac4e4
Compare
can you please update the demo ? |
Yes, I'm on it. |
Demo updated. I'll check at the travis issue now. |
a23e3fa
to
aa36d75
Compare
@adube please create a new issue for the CORS issue at https://github.com/camptocamp/c2cgeoportal/issues |
goog.require('ol.geom.Point'); | ||
goog.require('ol.layer.Tile'); | ||
goog.require('ol.source.OSM'); | ||
|
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.
missing goog.require('goog.asserts');
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.
I'm on it.
Looks good, please fix the missing require and this can be merged |
aa36d75
to
0650b8e
Compare
Fixed and ready for merge once Travis is happy. Thanks for the review. Issue about CORS created as well (see in the description above). |
@fredj Travis is happy |
This PR introduces the
gmf.EditFeature
service along with a simple example that demonstrates what you can do with it. The example should be replaced in the incoming days with one that uses the "coming soon" directives that allows the user to actually select an editable layer first.Same origin issues
When trying to insert, update or delete, we get errors:
Todo
Fix the "Same origin" issues, moved to: CORS issue with feature edit requests c2cgeoportal#2368Live example