Skip to content

Braze: Add connected content blocks names to sidebar [INTEG-2680] #9863

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 29, 2025

Conversation

joaquincasal
Copy link
Collaborator

Purpose

Add content block name to connected fields in the sidebar

Approach

Added a new app action function in order to retrieve content block names from Braze.

Testing steps

Create content block in Braze, refresh and check the sidebar contains both the fields names and the content blocks names.
Automated tests were added.

image

@joaquincasal joaquincasal requested a review from a team as a code owner May 27, 2025 21:20
Copy link
Collaborator

@JuliRossi JuliRossi left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -15,3 +21,25 @@ export function initContentfulManagementClient(context: FunctionEventContext): P
},
});
}

export async function getConfigAndConnectedFields(
Copy link
Collaborator

Choose a reason for hiding this comment

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

💡 This gave me an idea. Perhaps we can everything related to the configEntry to a configEntry service, so when we have to do the migration to appData is easier. We can leave it for a different PR

@joaquincasal joaquincasal merged commit 642de89 into braze-v2 May 29, 2025
11 checks passed
@joaquincasal joaquincasal deleted the braze-integ-2680 branch May 29, 2025 11:58
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