Skip to content
This repository was archived by the owner on Mar 25, 2021. It is now read-only.

Extract StringValueInput for better encapsulation#141

Merged
micgro42 merged 1 commit into
masterfrom
extractStringValueInput
Jan 13, 2021
Merged

Extract StringValueInput for better encapsulation#141
micgro42 merged 1 commit into
masterfrom
extractStringValueInput

Conversation

@micgro42
Copy link
Copy Markdown
Collaborator

@micgro42 micgro42 commented Jan 8, 2021

(This should be merged into master after #128 and is followed by #142)

This way the interfaces between StringValueInput and ItemValueLookup are nicely aligned which will make the next steps easier.

@micgro42 micgro42 requested review from Ladsgroup and guergana January 8, 2021 16:00
@micgro42 micgro42 force-pushed the explicitlyUnsetProperty branch 2 times, most recently from 46b901b to 6a908f6 Compare January 12, 2021 14:15
@micgro42 micgro42 force-pushed the extractStringValueInput branch from 47a5b47 to 9a7a184 Compare January 12, 2021 14:17
Base automatically changed from explicitlyUnsetProperty to master January 12, 2021 15:49
This way, the interfaces between StringValueInput and ItemValueLookup
are nicely aligned which will make the next step easier.
@micgro42 micgro42 force-pushed the extractStringValueInput branch from 9a7a184 to b93788f Compare January 13, 2021 10:27
Copy link
Copy Markdown
Contributor

@guergana guergana 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.

@micgro42 micgro42 merged commit b0da97c into master Jan 13, 2021
@micgro42 micgro42 deleted the extractStringValueInput branch January 13, 2021 11:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants