Skip to content

feat: create PushSource and catalogSource classes#30

Merged
y-lakhdar merged 5 commits into
mainfrom
LEN-839
May 25, 2023
Merged

feat: create PushSource and catalogSource classes#30
y-lakhdar merged 5 commits into
mainfrom
LEN-839

Conversation

@y-lakhdar

@y-lakhdar y-lakhdar commented May 19, 2023

Copy link
Copy Markdown
Contributor

Created PushSource and CatalogSource classes.
Both classes are not publicly exposed until the StreamService and PushService are implemented.
Once public, the Source class can be deprecated in favour of the new PushSource class.

Notable changes for these classes:

  • Users will now be able to instantiate catalog or push sources objects by providing the API url found in the Admin UI (source panel).
  • Sources will be marked as StreamEnabledSource or PushEnabledSource (or both) so they can use the appropriate upload service.

@y-lakhdar y-lakhdar requested review from a team, btaillon-coveo, hdhayneCoveo, louis-bompart and olamothe and removed request for a team May 19, 2023 15:38
@github-actions

github-actions Bot commented May 19, 2023

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Manifest Files

@y-lakhdar y-lakhdar requested a review from samisayegh May 19, 2023 15:39

@samisayegh samisayegh left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks great ✔️

Comment thread src/main/java/com/coveo/pushapiclient/ApiUrl.java Outdated
Comment thread src/main/java/com/coveo/pushapiclient/BaseSource.java Outdated
Comment thread src/test/java/com/coveo/pushapiclient/ApiUrlTest.java
Comment thread src/test/java/com/coveo/pushapiclient/ApiUrlTest.java Outdated
Comment thread src/test/java/com/coveo/pushapiclient/ApiUrlTest.java
Comment thread src/test/java/com/coveo/pushapiclient/ApiUrlTest.java Outdated
Comment thread src/main/java/com/coveo/pushapiclient/ApiUrl.java
Comment thread src/main/java/com/coveo/pushapiclient/ApiUrl.java
Comment thread src/main/java/com/coveo/pushapiclient/PushSource.java Outdated
Comment thread src/main/java/com/coveo/pushapiclient/CatalogSource.java
Comment thread src/main/java/com/coveo/pushapiclient/PushSource.java
@y-lakhdar y-lakhdar enabled auto-merge (squash) May 25, 2023 12:41
@y-lakhdar y-lakhdar merged commit 3a6b08a into main May 25, 2023
@y-lakhdar y-lakhdar deleted the LEN-839 branch May 25, 2023 12:42
y-lakhdar added a commit that referenced this pull request May 25, 2023
Update src/main/java/com/coveo/pushapiclient/ApiUrl.java

Co-authored-by: Benjamin Taillon <[email protected]>

add new getter to source classes

feat: create `PushSource` and `catalogSource` classes (#30)

https://coveord.atlassian.net/browse/LENS-839

update StreamService

add platformClient tests

add PowerMockito dependencies

remove unused import

remove unecessary method

downgrade build to Java 11

revert codeql deletion

lint

lint
This was referenced Apr 30, 2024
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.

3 participants