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

Document layer metadata search #7770

Merged
merged 4 commits into from Sep 20, 2022
Merged

Document layer metadata search #7770

merged 4 commits into from Sep 20, 2022

Conversation

elpaso
Copy link
Contributor

@elpaso elpaso commented Sep 8, 2022

Fix #7769

docs/user_manual/introduction/general_tools.rst Outdated Show resolved Hide resolved
docs/user_manual/introduction/general_tools.rst Outdated Show resolved Hide resolved
Comment on lines 1230 to 1234
The sources of metadata are implemented through a layer metadata provider
system that can be extended by plugins, QGIS provides out of the
box the layer metadata provider that retrieves the metadata from the
connections that allow for metadata storage (for more details
:ref:`here <savemetadatatodb>`).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Sorry but i find this sentence long, with too many "metadata" an do not understand it. Can we split it?
Also could be nice to replace the "here" by a more descriptive expression.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

better now?


.. _figure_metadata_save_options:

.. figure:: img/metadata_save_options.png
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wonder if a screenshot is necessary for these options. No strong opinion though.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@DelazJ feel free to apply the changes.

elpaso and others added 3 commits September 16, 2022 11:21
Co-authored-by: Harrissou Sant-anna <delazj@gmail.com>
Co-authored-by: Harrissou Sant-anna <delazj@gmail.com>
@DelazJ DelazJ merged commit d6a158e into qgis:master Sep 20, 2022
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.

Layer metadata provider gui (Request in QGIS)
2 participants