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

CIN-303 send content digest for alfresco insight connector #848

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

kamilbochno
Copy link
Collaborator

No description provided.

@kamilbochno kamilbochno marked this pull request as draft February 18, 2025 08:39
@PWS-TE PWS-TE self-requested a review February 19, 2025 07:12
@PWS-TE
Copy link
Collaborator

PWS-TE commented Feb 19, 2025

Good job, keep it up 🔥

@kamilbochno kamilbochno force-pushed the feature/CIN-303-send-content-digest-for-alfresco-insight-connector branch 5 times, most recently from 39c8765 to aecb63f Compare February 21, 2025 10:50
@kamilbochno kamilbochno force-pushed the feature/CIN-303-send-content-digest-for-alfresco-insight-connector branch from e0ccef1 to 92565b8 Compare February 21, 2025 11:05
 into feature/CIN-303-send-content-digest-for-alfresco-insight-connector
return versionNumber;
}
}
{}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why did we remove these methods?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Outdated, structure of file was changed

HxInsightRequest request = new HxInsightRequest(url, headers, null);
containerSupport.raiseRepoEvent(repoEvent);

// then
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are these comments necessary ? They just take up unnecessary space :)

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