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

removed the example on "numerical" filters, amended a couple of examples #86

Merged
merged 30 commits into from
Jun 19, 2023

Conversation

jrambla
Copy link
Contributor

@jrambla jrambla commented Jun 13, 2023

I've spotted a couple of places where examples seem incorrect.
Also removed the example on "numerical" filters as they do not actually exist.
I've only left the "alphanumerical" one.

mbaudis and others added 23 commits March 14, 2023 16:31
Some test changes for the new build workflow from website-docs
test edit
correction of a typo
Documentation: Additional contribution/development pages together with a reorganization of the navigation structure.

Also, clarification of the cohorts relation (#80).
More details and examples about the use of filters in various scenarios, including GET examples for age etc.

A main change here is the alignment of the filter types w/ the specification (e.g. no separate "numeric" type).
... and fixing `filterTerms` => `filteringTerms`
This is an attempt to get the documentation site towards "clean HTTPS"
* separate pages for records and handovers
* navigation restructuring

... more content to follow
@jrambla jrambla requested review from mbaudis and tb143 June 13, 2023 10:17
Edits have to be done in YAML and then propagated to JSON using bin/yamlerRunner.sh
@mbaudis
Copy link
Member

mbaudis commented Jun 14, 2023

@jrambla Thanks; I had tried to do this e.g. in fca3233 but may have missed some.

But you're now merging the documentation into main - which is fine in principle, but not strictly necessary & against the "main branch freeze". No problems from my side, though.

Note: one of he fixes directly affects framework, not only docs, so is a true fix in main (not affecting code, just inline examples). And this needs to be done the standard way (edit yaml, run bin/yamlrunner.sh OR manually edit both w/ diligence...) . Fixed in 23d49cc

@costero-e
Copy link
Collaborator

we need @tb143 approval here to close this PR

Copy link
Collaborator

@tb143 tb143 left a comment

Choose a reason for hiding this comment

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

Numerical filter operators are now subsumed under alphanumerical filter operators.

@costero-e costero-e merged commit d2be0e2 into main Jun 19, 2023
@costero-e costero-e added the documentation Improvements or additions to documentation label Jul 16, 2024
@costero-e costero-e added this to the 2.0.1 milestone Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants