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

editor: define custom documents typeahead type #350

Merged
merged 1 commit into from
Sep 3, 2020

Conversation

lauren-d
Copy link
Contributor

@lauren-d lauren-d commented Aug 28, 2020

Extends remote typeahead component to be able to search list
of suggestion of documents.

  • Adds a new class for custom documents typeahead.

Co-Authored-by: Lauren-D laurent.dubois@itld-solutions.be
Co-Authored-by: Renaud Michotte renaud.michotte@gmail.com

Why are you opening this PR?

  • Which task/US does it implement?
  • Which issue does it fix?

Dependencies

My PR depends on rero-ils#<xx>'s PR(s):

  • rero/rero-ils#

How to test?

  • What command should I have to run to test your PR?
  • What should I test through the UI?

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Extracted translations?

@lauren-d lauren-d self-assigned this Aug 28, 2020
@lauren-d lauren-d force-pushed the lau-custom-documents-typeahead branch 3 times, most recently from 677105e to 9c56780 Compare August 28, 2020 12:39
@lauren-d lauren-d force-pushed the lau-custom-documents-typeahead branch from 9c56780 to 72a9760 Compare August 28, 2020 12:49
@lauren-d lauren-d force-pushed the lau-custom-documents-typeahead branch from 72a9760 to f21ce2e Compare August 28, 2020 14:04
@lauren-d lauren-d force-pushed the lau-custom-documents-typeahead branch 2 times, most recently from 76d9147 to a71955a Compare August 29, 2020 08:11
Copy link
Contributor

@iGormilhit iGormilhit left a comment

Choose a reason for hiding this comment

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

The commit message is ok, but a small explanation on what this class does would be fine.

Extends remote typeahead component to be able to search list
of suggestion of documents.

* Adds a new class for custom documents typeahead.

Co-Authored-by: Lauren-D <laurent.dubois@itld-solutions.be>
Co-Authored-by: Renaud Michotte <renaud.michotte@gmail.com>
@lauren-d lauren-d force-pushed the lau-custom-documents-typeahead branch from a71955a to eba11fe Compare August 31, 2020 07:20
Copy link
Contributor

@iGormilhit iGormilhit left a comment

Choose a reason for hiding this comment

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

Commit message approved.

@iGormilhit iGormilhit added this to the v0.13.0 (rero-ils) milestone Sep 1, 2020
@benerken benerken self-requested a review September 3, 2020 11:22
@lauren-d lauren-d merged commit ed90815 into rero:dev Sep 3, 2020
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.

6 participants