-
Notifications
You must be signed in to change notification settings - Fork 0
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
Alignment df7 10 Roads #4
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, found just two minor things that should be fixed before merging the PR.
def geplannteMaßnahmen = c.Maßnahmen.geplante.values() | ||
def geplannteMaßnahmenNutzen = c.Maßnahmen.nutzen.value() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
geplannte
has an addtional n
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, for the hint. I fixed it now.
publicConsultation_reviewExplanation( c.Öffentlichkeitsbeteiligung.lapUeberarbeitetKommentar.value() ) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code style: wrong indentation of line 518
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't what was the issue here, because for me it looked correct in the groovy editor in hale studio.
But I removed and added the indentation with an editor and now it should be correct.
In case no planned measure is set and 'c.Maßnahmen.nutzen.value()' is empty a default value should be set. The respective changes are done here. SVC-1696
In case 'c.Öffentlichkeitsbeteiligung.lapUeberarbeitet.value()' is set to 'Nein' in the source data and 'c.Öffentlichkeitsbeteiligung.lapUeberarbeitetKommentar.value()' is empty a default value should be set for 'NoiseActionPlanMajorRoad.publicConsultation_reviewExplanation'. The respective changes are done here. SVC-1696
SVC-1696
e85d163
to
77075e2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
SVC-1696