Skip to content
This repository has been archived by the owner on Oct 26, 2023. It is now read-only.

Fix styling of sessions in hallway and exhibition #42

Merged

Conversation

svenreimers
Copy link
Contributor


^Add meaningful description above

By opening a pull request you confirm that, unless explicitly stated otherwise, the changes -

  • are all your own work, and you have the right to contribute them.
  • are contributed solely under the terms and conditions of the Apache License 2.0 (see section 5 of the license for more information).

Please make sure (eg. git log) that all commits have a valid name and email address for you in the Author field.

If you're a first time contributor, see the Contributing guidelines for more information.

@svenreimers svenreimers requested a review from mklaehn October 2, 2023 17:10
Copy link
Contributor

@mklaehn mklaehn left a comment

Choose a reason for hiding this comment

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

Seems like some changes were committed that shouldn't have been committed...

// jvmArgs '-Dorg.tweetwallfx.scheduledata.day=friday'
// jvmArgs '-Dorg.tweetwallfx.scheduledata.time=10:00Z'
// jvmArgs '-Dorg.tweetwallfx.conference.randomRatedTalks=true'
jvmArgs '-Dorg.tweetwallfx.scheduledata.day=thursday'
Copy link
Contributor

Choose a reason for hiding this comment

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

Undo please

@@ -2,6 +2,7 @@
"conferenceClient": {
"eventBaseUri": "https://dvbe23.cfp.dev/api/public",
"votingResultsUri": "https://us-central1-voxxrin-v3.cloudfunctions.net/publicEventStats",
"votingResultsToken": "eventStats:devoxx-voxxed:c5ac7b45-f7cb-4192-a0ee-f6f7596cad30",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should no longer be necessary. Undo please

@@ -112,6 +112,9 @@
"nodeSelector": "#scheduleNode"
}
},
{
"stepClassName": "org.tweetwallfx.conference.stepengine.steps.SpeakerImageMosaicStep"
Copy link
Contributor

Choose a reason for hiding this comment

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

Time to go to production?

@@ -29,8 +29,8 @@ application {
}

tasks.withType(JavaExec).configureEach {
// jvmArgs '-Dorg.tweetwallfx.scheduledata.day=friday'
// jvmArgs '-Dorg.tweetwallfx.scheduledata.time=10:00Z'
jvmArgs '-Dorg.tweetwallfx.scheduledata.day=monday'
Copy link
Contributor

Choose a reason for hiding this comment

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

Undo please

@@ -29,8 +29,8 @@ application {
}

tasks.withType(JavaExec).configureEach {
// jvmArgs '-Dorg.tweetwallfx.scheduledata.day=friday'
// jvmArgs '-Dorg.tweetwallfx.scheduledata.time=10:00Z'
jvmArgs '-Dorg.tweetwallfx.scheduledata.day=monday'
Copy link
Contributor

Choose a reason for hiding this comment

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

Undo please

@mklaehn
Copy link
Contributor

mklaehn commented Oct 2, 2023

Seems like some changes were committed that shouldn't have been committed...

Also spotless is complaining

@svenreimers svenreimers force-pushed the bugfix/session-title-styling branch from 68caef8 to db11b28 Compare October 2, 2023 18:34
@svenreimers svenreimers merged commit 1d53845 into TweetWallFX:main Oct 2, 2023
6 checks passed
@svenreimers svenreimers deleted the bugfix/session-title-styling branch October 2, 2023 19:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants