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

many: backport some usefull functions #20

Merged
merged 1 commit into from
Oct 16, 2019
Merged

Conversation

jma
Copy link
Contributor

@jma jma commented Oct 12, 2019

  • Adds new test script.
  • Improves test by adding tester app into the tests.
  • Adds new form validators: to test remotely the uniqueness of a value,
    to check a range of time.
  • Adds a function to clean a json before backend POST: remove empty
    values.
  • Improves the valueAlreadyExists function of the record-service to use
    getRecords instead of a direct http call.
  • Add variables to record-service for default and max number of results.
    Co-Authored-by: Johnny Mariéthoz Johnny.Mariethoz@rero.ch
    Co-Authored-by: Bertrand Zuchuat bertrand.zuchuat@rero.ch

* Adds new test script.
* Improves test by adding tester app into the tests.
* Adds new form validators: to test remotely the uniqueness of a value,
to check a range of time.
* Adds a function to clean a json before backend POST: remove empty
values.
* Improves the valueAlreadyExists function of the record-service to use
getRecords instead of a direct http call.
* Add variables to record-service for default and max number of results.
Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
Co-Authored-by: Bertrand Zuchuat <bertrand.zuchuat@rero.ch>
@jma jma force-pushed the maj-backport-fn branch from 92b8c2c to 850f41a Compare October 12, 2019 07:10
@sebdeleze sebdeleze merged commit 852a22f into rero:dev Oct 16, 2019
@jma jma deleted the maj-backport-fn branch November 21, 2019 16:09
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.

2 participants