Skip to content

Conversation

@bhharsh13
Copy link
Contributor

@bhharsh13 bhharsh13 commented Jan 16, 2025

Implement ready only and edit interfaces on an Integer field.

https://ikmdev.atlassian.net/browse/IIA-1228

Please find the attached screenshots of the changes validated on local.
existing Integer Value
Clicking on edit to update the integer value to 10 and clicking on Submit button.
updating to new Integer value

Close the Pattern and re-open the Semantic Pattern window reflecting the updated Integer value.
updated the Integer Field values

@dukke
Copy link
Contributor

dukke commented Jan 16, 2025

Hi Bhavani @bhharsh13 !
Great work!

Can you possibly tweak the title of the PR. We usually follow this pattern:

  • "Feature" or "Bugfix" depending on whether it is a bug or a feature
  • After that insert "/" and then the issue id

So, this would look like: "Feature/IIA-1228: Implement ready only and edit interfaces on an Integer field"

Another thing worth adding is a link to the jira ticket in the issue description.

Thanks! :)

@bhharsh13 bhharsh13 changed the title Implement ready only and edit interfaces on an Integer field. Feature/IIA-1228: Implement ready only and edit interfaces on an Integer field. Jan 16, 2025
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.

Nice @bhharsh13
All looks good to me just had that minor comment.

@bhharsh13
Copy link
Contributor Author

Closing the PR due to some more amendments to other's PR which are dependent

@bhharsh13 bhharsh13 closed this Jan 16, 2025
@bhharsh13 bhharsh13 reopened this Jan 16, 2025
@bhharsh13
Copy link
Contributor Author

Closing this PR due to change in the implementation strategy.

@bhharsh13 bhharsh13 closed this Jan 17, 2025
@bhharsh13 bhharsh13 deleted the feature/IIA-1228-Integer-Data-Type branch January 17, 2025 17:06
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.

2 participants