Skip to content

Conversation

@swaroopsalvi
Copy link
Contributor

@swaroopsalvi swaroopsalvi commented Aug 25, 2025

image image

Before Change:
image

After Change:
image

Online User:

image

Select DB
image

Login using KOMET user
image

image image image

swsalvi and others added 5 commits August 22, 2025 17:44
…ished/IIA-2623-Implement-Author-Attribution-in-Concept-Window-Concept-Version-Form

# Conflicts:
#	kview/src/main/java/dev/ikm/komet/kview/mvvm/view/journal/JournalController.java
…uthor-Attribution-in-Concept-Window-Concept-Version-Form
Copy link
Contributor

@dukke dukke left a comment

Choose a reason for hiding this comment

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

@swaroopsalvi I left a small remark.

Also I'm not sure but when the stamp form hasn't been changed yet should we show the last stamp author rather than the current logged in user.
For the date we show the last stamp date and only when there's a change in the form do we change the date, in this case to "Uncommited".
In other words only when there's a change in the form and it's ready to be submitted do we show the logged in user as the author... I had the idea this was the case, but I could be wrong.

Thanks!

@swaroopsalvi
Copy link
Contributor Author

@swaroopsalvi I left a small remark.

Also I'm not sure but when the stamp form hasn't been changed yet should we show the last stamp author rather than the current logged in user. For the date we show the last stamp date and only when there's a change in the form do we change the date, in this case to "Uncommited". In other words only when there's a change in the form and it's ready to be submitted do we show the logged in user as the author... I had the idea this was the case, but I could be wrong.

Thanks!

Makes sense. Fixed it.

Copy link
Contributor

@dukke dukke left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks Swaroop!

Copy link
Contributor

@jdsmithsos jdsmithsos left a comment

Choose a reason for hiding this comment

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

looks good

@swaroopsalvi swaroopsalvi merged commit 05b2bd0 into ikmdev:main Aug 26, 2025
7 checks passed
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.

4 participants