Skip to content

OBPIH-6698 Include label and defaultMessage in base field componnt when props are passed as dynamic attributes#4920

Merged
awalkowiak merged 1 commit intodevelopfrom
feature/OBPIH-6698-fix-passing-label-props-in-render-form0-util
Oct 30, 2024
Merged

OBPIH-6698 Include label and defaultMessage in base field componnt when props are passed as dynamic attributes#4920
awalkowiak merged 1 commit intodevelopfrom
feature/OBPIH-6698-fix-passing-label-props-in-render-form0-util

Conversation

@drodzewicz
Copy link
Collaborator

@drodzewicz drodzewicz commented Oct 30, 2024

✨ Description of Change

A concise summary of what is being changed. Please provide enough context for reviewers to be able to understand the change and why it is necessary. If the issue/ticket already provides enough information, you can put "See ticket" as the description.

Link to GitHub issue or Jira ticket: OBPIH-6698

Description:
After some recent changes to the receiving workflow it appears that props label and defaultMessage were not properly passed to the BaseField component when passing them through dyanmicAttributes which broke the test because screen reader was not able to find the appropriate textfield when looking for Receving now field by label


📷 Screenshots & Recordings (optional)

If this PR contains a UI change, consider adding one or more screenshots here or link to a screen recording to help reviewers visualize the change. Otherwise, you can remove this section.

@drodzewicz drodzewicz self-assigned this Oct 30, 2024
@github-actions github-actions bot added type: feature A new piece of functionality for the app domain: frontend Changes or discussions relating to the frontend UI labels Oct 30, 2024
@awalkowiak awalkowiak merged commit c0f6b8e into develop Oct 30, 2024
@awalkowiak awalkowiak deleted the feature/OBPIH-6698-fix-passing-label-props-in-render-form0-util branch October 30, 2024 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: frontend Changes or discussions relating to the frontend UI type: feature A new piece of functionality for the app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants