4.8.6.1 cert gen program - #29
Open
karthik-tarento wants to merge 92 commits into
Open
Conversation
…e event generation
…e event generation
…e event generation -Kafka event changes
…e event generation -pom.xml changes
…e event generation -Added the code changes for Adding logic
…e event generation -Added the changes for batches Validation and Enrolment Validation
…e event generation Made the changes for the batches Read
* Initial Commit 1. postPublicRelationUpdateStream Added Kafka consumer and producer. 2. Added the configuration in the PostPublishProcessorConfig. 3. Added service method postPublishRelationUpdate to the BatchCreation. 4. Added a function classs PostPublishRelationUpdaterFunction. 5. Added the configuration in the Post-publish-processor.conf * Code changes Code changes * Dev #104434-Enhance post publish processor flink job and add parentCollections details against Course. Code Review fixes. * Added method to get course from redis * Dev #104434- Enhance post publish processor flink job and add parentCollections details against Course. Code Fixes * Dev #104434-Enhance post publish processor flink job and add parentCollections details against Course Content State Url Updated. * Updated code. Using proper system update API * Optimized code * Formatted code * Formatted code * reading keyspace from config * reading keyspace from config --------- Co-authored-by: tarentomaheshvakkund <[email protected]>
…e event generation -pom.xml changes for the job-config
…osterImage (#28) 1. In the getCourseInfo() , in the contentReadApi added field posterImage to retrieve the posterImage details. Added the coursePosterImage in the Map 2. Added constants int he CollectionCerPrePorcessorConfig - posterImage. 3.CertificateGeneratorFunction - In the context.output added new fields in the NotificationMetaData call - coursePosterImage and providerName. 4.NotificationMetaData class added new fields - coursePosterImage and providerName. 5.Added constants int he CertificateGeneratorConfig - courseProvider, coursePosterImage. 6. Added empty string in the test class NotifierFunctionTest
New Email Template - endpoint added for email notification
…ts for program type courses 1. Added a method verifyPrimaryCategory() to check if the primary category is blended program, curated program or program and skip the event accordingly.
…b/knowledge-platform-jobs into 4.8.6.1-certGenProgram
…imaryCategory and parentCollections
…b/knowledge-platform-jobs into 4.8.6.1-certGenProgram
ArpithaSureshappa
pushed a commit
to ArpithaSureshappa/knowledge-platform-jobs
that referenced
this pull request
Oct 21, 2024
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
4 participants
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.
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Type of change
Please choose appropriate options.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes in the below checkboxes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist: