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.

Sign up for GitHub

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

Jump to bottom

docs: update query documentation for DESC date (MAPCO-2922) #68

Merged
merged 4 commits into from
Dec 7, 2023
Merged

docs: update query documentation for DESC date (MAPCO-2922) #68

merged 4 commits into from
Dec 7, 2023

Conversation

asafMasa
Copy link
Contributor

No description provided.

@asafMasa asafMasa added the documentation Improvements or additions to documentation label Nov 14, 2023
@asafMasa asafMasa requested a review from shimoncohen November 14, 2023 09:50
shimoncohen
shimoncohen previously approved these changes Nov 19, 2023
View reviewed changes
@shimoncohen
Copy link
Contributor

@asafMasa Please make the following changes:

  1. Reorder the examples such that the basic one is first.
  2. Make all of the parameter text bold

@asafMasa asafMasa changed the title docs: update query documentation for DESC date docs: update query documentation for DESC date (MAPCO-2922) Nov 27, 2023
@asafMasa
Copy link
Contributor Author

@asafMasa Please make the following changes:

  1. Reorder the examples such that the basic one is first.
  2. Make all of the parameter text bold

@shimoncohen Fixed

shimoncohen
shimoncohen approved these changes Dec 7, 2023
View reviewed changes
docs/ogc/protocols/examples/ogc-csw-query-examples.md Outdated
Response will contain up to 100 records ('maxRecords="100"') that their **productType** is **'Orthophoto'**, **maxResolutionDeg** is less than **'5.36441E-06'** ordered by DESC **ingestionDate** after **'2023-10-15T00:00:01Z'**
Copy link
Contributor

Choose a reason for hiding this comment

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

DESC is still not in bold even though it's an integral part of the query.

@shimoncohen shimoncohen merged commit 14712b3 into master Dec 7, 2023
@shimoncohen shimoncohen deleted the addDescIngestionDateQuery branch December 7, 2023 08:05
@mapcolonies-devops mapcolonies-devops mentioned this pull request Nov 14, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shimoncohen shimoncohen shimoncohen approved these changes

Assignees

@asafMasa asafMasa

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@asafMasa @shimoncohen @asafmas-rnd