Skip to content

Fix: Add remoteAuthS3 type to DiscussionEmbedConfig Interface#159

Merged
chrisjtang merged 6 commits into
masterfrom
sso-type
Jul 7, 2025
Merged

Fix: Add remoteAuthS3 type to DiscussionEmbedConfig Interface#159
chrisjtang merged 6 commits into
masterfrom
sso-type

Conversation

@chrisjtang
Copy link
Copy Markdown
Contributor

@chrisjtang chrisjtang commented Jun 30, 2025

Description

These changes add the remoteAuthS3 type to the DiscussionEmbedConfig interface.

Motivation and Context

#151

How Has This Been Tested?

Using https://github.com/chrisjtang/foodandstuff/tree/typescript, the original bug was reproduced and fixed with the change in this PR.

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)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
    • I have updated the documentation accordingly.
  • All new and existing tests passed.

@chrisjtang chrisjtang self-assigned this Jun 30, 2025
@chrisjtang chrisjtang requested review from Hovspian and tail July 1, 2025 19:38
Comment thread tests/DiscussionEmbed.js Outdated
Copy link
Copy Markdown

@Hovspian Hovspian left a comment

Choose a reason for hiding this comment

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

lgtm

@chrisjtang chrisjtang merged commit 99dc890 into master Jul 7, 2025
3 checks passed
@chrisjtang chrisjtang deleted the sso-type branch July 7, 2025 13:28
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