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

Update Template #8

Merged
merged 16 commits into from
Sep 28, 2023
Merged

Update Template #8

merged 16 commits into from
Sep 28, 2023

Conversation

jasperkamerling
Copy link
Collaborator

No description provided.

@jasperkamerling jasperkamerling changed the title Feature/update Update Template Sep 28, 2023
Signed-off-by: Jasper Kamerling <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
@@ -14,7 +15,7 @@ import kotlin.random.Random

@Service
class GxfKafkaProducer(
private val kafkaTemplate: KafkaTemplate<String, Any>,
private val kafkaTemplate: KafkaTemplate<String, Measurement>,
Copy link
Member

Choose a reason for hiding this comment

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

Voor KOD zullen we waarschijnlijk meerdere producers krijgen, nl. voor meet-, status- en alarmberichten (en mogelijk ook nog andere zoals bijv registratie- en configuratieberichten van domain naar protocol)
Is Measurement dan voldoende dekkend?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Het dient alleen als voorbeeld. ik doe verder niks er mee

import com.gxf.service.Measurement
import com.gxf.service.VoltageMeasurement
import org.gxf.service.Measurement
import org.gxf.service.VoltageMeasurement
Copy link
Member

Choose a reason for hiding this comment

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

VoltageMeasurement is wel erg toegespitst op LSMeten.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Het is alleen om als voorbeeld te dienen. Ik wil in deze module niet meer dan 1 objectje.

kotlin("jvm") version "1.9.10" apply false
kotlin("plugin.spring") version "1.9.10" apply false
kotlin("plugin.jpa") version "1.9.10" apply false
id("com.github.davidmc24.gradle.plugin.avro") version "1.8.0" apply false
Copy link
Member

Choose a reason for hiding this comment

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

Hoort deze plugin niet enkel in de avro component thuis?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

klopt, met apply false zeggen we dat we het niet toepassen, alleen een versie defineren.

Copy link
Member

Choose a reason for hiding this comment

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

Dat snap ik, mijn onderliggende issue was hierbij dat, als we een service gaan implementeren zonder gebruik te maken van Avro, we op meerdere plekken code moeten verwijderen...
(Is waarschijnlijk niet onder uit te komen, als je versiebeheer op een plek wil hebben...)

jasperkamerling and others added 7 commits September 28, 2023 13:07
Co-authored-by: Sander Verbruggen <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
…/configuration/oauth/OAuthAuthenticateCallbackHandler.kt

Co-authored-by: Sander Verbruggen <[email protected]>
Signed-off-by: Jasper Kamerling <[email protected]>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@jasperkamerling jasperkamerling merged commit ab6217c into main Sep 28, 2023
@jasperkamerling jasperkamerling deleted the feature/update branch September 28, 2023 14:46
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