Skip to content
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

SnsTemplate provides functionality to validate topic existence (#1137) #1142

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

hardikSinghBehl
Copy link

@hardikSinghBehl hardikSinghBehl commented Apr 28, 2024

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Implemented the method boolean topicExists(String topicArn) in SnsTemplate to validate the existence of an SNS topic via an ARN.

💡 Motivation and Context

Implementation for issue #1137

💚 How did you test it?

The functionality has been tested by writing integration tests for the new method.

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • I updated reference documentation to reflect the change
  • All tests passing
  • No breaking changes

🔮 Next steps

@github-actions github-actions bot added the component: sns SNS integration related issue label Apr 28, 2024
@MatejNedic
Copy link
Member

Hey @hardikSinghBehl thanks on PR! Will take a deeper look tomorrow and come back to you

Copy link
Member

@MatejNedic MatejNedic left a comment

Choose a reason for hiding this comment

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

All good, just remove that 1 thing from gitignore

.gitignore Outdated Show resolved Hide resolved
@hardikSinghBehl
Copy link
Author

done, @MatejNedic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: sns SNS integration related issue type: documentation Documentation or Samples related issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants