Skip to content

Snyder default info#105

Closed
brenthosie wants to merge 7 commits intomasterfrom
snyder-default-info
Closed

Snyder default info#105
brenthosie wants to merge 7 commits intomasterfrom
snyder-default-info

Conversation

@brenthosie
Copy link
Member

@brenthosie brenthosie commented May 23, 2025

Description

#91 was opened by @jonsnyder in 2023. Rather than having @jonsnyder resolve the conflicts, I've done that here on his behalf.

Motivation and Context

This adds a new api endpoint "/defaultInitInfo/:type/:name" which returns the default init info for a component. This api endpoint calls a user specified callback to modify the initInfo, getDefaultInitInfo. This is useful for generating and setting an access token or setting up custom defaults for extensionSettings or other properties in the initInfo.

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Closes #91

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@brenthosie brenthosie force-pushed the snyder-default-info branch from 22d164d to 146a2c0 Compare May 23, 2025 16:30
@brenthosie
Copy link
Member Author

@jonsnyder please advise on testing scenarios for this branch

Brent Hosie added 2 commits May 23, 2025 11:16
@brenthosie brenthosie force-pushed the snyder-default-info branch from 146a2c0 to a2bd1b2 Compare May 23, 2025 19:49
@brenthosie
Copy link
Member Author

handled by #109

@brenthosie brenthosie closed this Oct 4, 2025
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