Querying members with opensearch#1001
Merged
epipav merged 10 commits intoenhancement/opensearch-C-1317from Jun 20, 2023
Merged
Conversation
themarolt
reviewed
Jun 20, 2023
…rch-querying-members-C-1317
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes proposed ✍️
What
🤖 Generated by Copilot at fff2e58
This pull request integrates OpenSearch with the backend services and the API of the crowd.dev platform. It adds the necessary dependencies, configuration, and middleware to connect to an OpenSearch node and use it for search operations. It also removes some unused and outdated files and variables related to the database attributes and the SQS queue. Additionally, it updates the test environment setup and the README file to reflect the changes.
🤖 Generated by Copilot at fff2e58
Why
How
🤖 Generated by Copilot at fff2e58
@crowd/opensearchpackage as a local dependency of the backend services, and the@opensearch-project/opensearchpackage as a dependency of the@crowd/opensearchpackage (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link)CROWD_OPENSEARCH_NODEenvironment variable to thebackend/.env.dist.composedandbackend/.env.dist.localfiles to specify the URL of the OpenSearch node (link, link)CROWD_SQS_PORTenvironment variable to thebackend/.env.testfile to specify the port of the SQS service (link)opensearchconfiguration to thebackend/config/custom-environment-variables.jsonfile to map the environment variables to the properties of the OpenSearch client (link)sqsandopen-search-testservices to thebackend/docker-compose.test.yamlfile to create the SQS and OpenSearch containers for the test environment (link)IOpenSearchConfiginterface to thebackend/src/conf/configTypes.tsfile to define the type of the opensearch configuration (link)IOpenSearchConfiginterface and theOPENSEARCH_CONFIGconstant from thebackend/src/conf/index.tsfile to make them available for other modules (link, link)getOpensearchClientfunction from the@crowd/opensearchpackage and theOPENSEARCH_CONFIGconstant from the../confmodule to thebackend/src/api/index.tsfile, and use them to create an instance of the OpenSearch client (link, link)opensearchMiddlewarefunction from the../middlewares/opensearchMiddlewaremodule to thebackend/src/api/index.tsfile, and use it to bind the OpenSearch client to the request object of the API service (link, link)CROWD_SQS_INTEGRATION_RUN_WORKER_QUEUEenvironment variable from thebackend/.env.dist.composedandbackend/.env.dist.localfiles to avoid confusion and duplication (link, link)loggingproperty to the database configuration in thebackend/config/custom-environment-variables.jsonfile to enable or disable the logging of the database queries (link)AttributeDatainterface to extend theIMemberAttributeinterface from the@crowd/typespackage in thebackend/src/database/attributes/attribute.tsfile to reuse the common properties of the member attribute type (link)backend/src/database/attributes/member/crowd.ts,backend/src/database/attributes/member/devto.ts,backend/src/database/attributes/member/enums.ts,backend/src/database/attributes/member/github.ts,backend/src/database/attributes/member/hackerNews.ts,backend/src/database/attributes/member/linkedin.ts,backend/src/database/attributes/member/reddit.ts, andbackend/src/database/attributes/member/stackOverflow.tsas they are no longer needed (link, link, link, link, link, link, link, link)Checklist ✅
Feature,Improvement, orBug.