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

13 optimize search experience #37

Merged
merged 31 commits into from
Sep 2, 2021
Merged

13 optimize search experience #37

merged 31 commits into from
Sep 2, 2021

Conversation

DaTebe
Copy link
Collaborator

@DaTebe DaTebe commented Sep 2, 2021 •
edited
Loading

Quick checklist for code changes:

  • Created a new branch based on dev
  • This PR will be submitted back in dev
  • All commits correspond to our convention
  • The code linted
  • All checks passed
  • README's updated (if configuration, behavior changed)
  • OpenAPI specs updated (if API changed)
  • Versions for changed components incremented (in the component specific file and in corresponding docker-compose.yml)
  • Any breaking changes clearly communicated to the core team and do not imply system dysfunction
  • Documentation update proposed (if configuration, behavior, structure, processes changed)

DaTebe and others added 30 commits August 23, 2021 18:08
@DaTebe
feat(SA): more natural way of searching as default
1bbc35f
@DaTebe
fix(SA): removed unneeded console logging
7178c0f
@DaTebe
chore(docker): changed stop to down
0a30f13
@DaTebe
feat(esc): mapping test
bb2dfab
@setaman
feat(common): finalize json schema to ES mapping utility
6275384
@setaman
feat(common): add json schema convert utility
8bd3f54
@DaTebe
chore(common): json schema strict mode to false
a5dcb18
@DaTebe
chore(common): moved json converter to parser folder
da1f4d2
@DaTebe
feat(ESC): set mapping for every index used
71feaf4
@DaTebe
feat(ESC): create index with mapping if not exists
907ca2d
@DaTebe
fix(ESC): load dereferenced schema to build mapping
8246507
@DaTebe
fix: no analyzer on keyword type ES
d94a746
@DaTebe
feat(schema): added elasticsearch options to user
e9cee2c
@DaTebe
feat(schemata): added elasticsearch config to entity
a2a0a41
@DaTebe
fix: json to es parser added date to not have analyzer
7dd1e86
@DaTebe
feat: implemented better json schema to es mapping condition
b58d02a
@DaTebe
feat: updated common in all services + version bump
1432aef
@DaTebe
feat(ESC): error handling in root async method
132fe22
@DaTebe
feat(elasticsearch): added mapping for reviews
96930e3
@DaTebe
feat(elasticsearch): added asset mappings
0880921
@DaTebe
feat(elasticsearch): added diva custom settings for indices
a879bd6
@DaTebe
feat(elasticsearch): added mapping info to commonProperties
2e5ae36
@DaTebe
feat(ESC): now also sets settings for indices
7de31f6
@DaTebe
fix(elasticsearch): fixed user schema mapping
8916f28
@DaTebe
chore: added comments to json -> es parser
64b8400
@DaTebe
fix(schema): removed indexing option for now
aaaec2c
@DaTebe
feat(schema): added resource es mappings
4a18632
@DaTebe
feat(schema): added es mapping for all schemata
10e900e
@setaman
fix(common): handle properly definitions on json schema conversion
e5e484d
@setaman
Merge remote-tracking branch 'origin/13-optimize-search-experience' i…
2e3d952 
…nto 13-optimize-search-experience
@setaman setaman self-requested a review September 2, 2021 13:48
@DaTebe DaTebe merged commit f7871dd into dev Sep 2, 2021
@DaTebe DaTebe deleted the 13-optimize-search-experience branch September 7, 2021 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@setaman setaman setaman approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DaTebe @setaman