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

MS Teams Upload attachment feature #421

Closed

Conversation

vaibhav-db
Copy link
Contributor

No description provided.

vaibhav-db and others added 5 commits November 21, 2023 17:15
…ing bot team framework

Merge in SYMPHONYP/symphony-java-toolkit from feature/SYMPHONYP-1106-download-feature-in-spring-bot-team-framework to spring-bot-master-db

* commit '15451c97348710774d7276e65acacbba9271d0f1':
  SYMPHONYP-1106 upload attachment feature in famework
  Fix for finos#342
  update pom
  Update CHANGES.md
  Update CHANGES.md
@vaibhav-db vaibhav-db self-assigned this Apr 29, 2024
Copy link

linux-foundation-easycla bot commented Apr 29, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link
Member

@robmoffat robmoffat left a comment

Choose a reason for hiding this comment

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

Looking good so far!

import org.finos.springbot.workflow.java.mapping.ChatHandlerExecutor;

/**
* Get chat button detail in controller
Copy link
Member

Choose a reason for hiding this comment

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

Can you describe what this is for?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As we discussed last time, we need button information on controller method. For that we have added this class.

import com.microsoft.bot.schema.teams.FileConsentCardResponse;
import com.microsoft.bot.schema.teams.FileInfoCard;

public class FileActivityHandler extends MessageActivityHandler {
Copy link
Member

Choose a reason for hiding this comment

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

Please check the code coverage as a result of adding this. I have a feeling there will need to be some unit tests added to make sure this works.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, I will check the code coverage and also add the unit testcases.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

committed testcases..

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @robmoffat ,

Now code coverage is 72.53%

Copy link
Member

Choose a reason for hiding this comment

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

Merge in SYMPHONYP/symphony-java-toolkit from feature/SYMPHONYP-1106-download-feature-in-spring-bot-team-framework to spring-bot-master-db

* commit '9129c834f6119ed03b861233bb6c6dc1e1ff4b08':
  SYMPHONYP-1118 test case for File handler
…ing bot team framework

Merge in SYMPHONYP/symphony-java-toolkit from feature/SYMPHONYP-1106-download-feature-in-spring-bot-team-framework to spring-bot-master-db

* commit '4958a388f0b9aa305b75a40a0beddee0275de69c':
  SYMPHONYP-1118 test case for File handler
  SYMPHONYP-1118 test case for File handler
  SYMPHONYP-1118 test case for File handler
@robmoffat robmoffat self-requested a review May 22, 2024 09:38
…ing bot team framework

Merge in SYMPHONYP/symphony-java-toolkit from feature/SYMPHONYP-1106-download-feature-in-spring-bot-team-framework to spring-bot-master-db

* commit '224316daf824e9ee8ffb34a1e7583a2e797605e4':
  Release note
  Release note
@robmoffat robmoffat closed this Jun 5, 2024
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.

3 participants