ocpp-v1_6: BootNotificationRequest: Allow optional properties to be set null

[bantu]

Fix #221

[TVolden]

Looks good. The copy right year should be bumped up to 2022

[bantu]

@TVolden Updated

[TVolden]

It's weird. The maven build failed because of some java doc.

[TVolden]

Maybe try adding the target source to the pom.xml file:
https://stackoverflow.com/questions/58836862/jdk-11-and-javadoc

--- a/ocpp-v1_6/pom.xml
+++ b/ocpp-v1_6/pom.xml
@@ -116,6 +116,9 @@
             <plugin>
               <groupId>org.apache.maven.plugins</groupId>
               <artifactId>maven-javadoc-plugin</artifactId>
+              <configuration>
+                <source>8</source>
+              </configuration>
               <version>2.9.1</version>
               <executions>
                 <execution>

[bantu]

@TVolden Added

[bantu]

Are we seeing this here? https://stackoverflow.com/a/74185633

[bantu]

Are we seeing this here? https://stackoverflow.com/a/74185633

Possible. The version seems to match.

 Java configuration:
  Distribution: temurin
  Version: 11.0.17+8
  Path: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/11.0.17-8/x64

[TVolden]

It seems so.

Command line was: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/11.0.17-8/x64/bin/javadoc @options @packages

So how to fix it?

[bantu]

I've removed my not-working fix commit from this branch.

So how to fix it?

Can we downgrade to Temurin 11.0.16 or upgrade to a fixed version?

[bantu]

Is the javadoc output actually deployed or used somewhere?

[bantu]

Can we downgrade to Temurin 11.0.16 or upgrade to a fixed version?

Added commit doing that.

[TVolden]

It worked, amazing work @bantu.