Skip to content

Feature/finished/iia 1130 rc anon#355

Merged
carldea merged 3 commits intoikmdev:mainfrom
dholubek:feature/finished/IIA-1130-rc-anon
Apr 4, 2025
Merged

Feature/finished/iia 1130 rc anon#355
carldea merged 3 commits intoikmdev:mainfrom
dholubek:feature/finished/IIA-1130-rc-anon

Conversation

@dholubek
Copy link
Contributor

@dholubek dholubek commented Apr 2, 2025

This is a follow up PR for https://ikmdev.atlassian.net/browse/IIA-1130

We are using anonymous concept. This will likely affect the JavaFX UI components

image

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.

Given yesterdays talk with Keith and our talk with Ryan at the architecture call we'll likely need to discuss this further to reach a consensus on what our approach will be.
I think our understanding of autosave and saving non valid (empty) fields has changed.
There are a few approaches that Keith mentioned we can take...we'll need to decide on one.

Copy link
Contributor

@carldea carldea 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! thank you.

@carldea carldea merged commit 7be47a0 into ikmdev:main Apr 4, 2025
6 checks passed
@carldea
Copy link
Contributor

carldea commented Apr 4, 2025

Given yesterdays talk with Keith and our talk with Ryan at the architecture call we'll likely need to discuss this further to reach a consensus on what our approach will be. I think our understanding of autosave and saving non valid (empty) fields has changed. There are a few approaches that Keith mentioned we can take...we'll need to decide on one.

I think it was decided to use the Anonymous concept for now. I hope this doesn't cause more confusion. Let's just get the happy path first (initial create).

@dholubek dholubek deleted the feature/finished/IIA-1130-rc-anon branch May 29, 2025 16:23
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