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

Migrate to Adaptive Card 1.3 Version #243

Merged
merged 4 commits into from
Oct 28, 2024

Conversation

Sakthivel-Ramasamy
Copy link

Code changes commit for webexpythonsdk's Adaptive Card 1.1 to 1.3 version upgrade. Changes involve code files restructuring inline with Adaptive Card's official documentation (https://adaptivecards.io/explorer/).

Sakthivel Ramasamy (sakthram) added 2 commits September 1, 2024 08:46
Remove Adaptive Card Version 1.1 codes as some code refactoring is done to make way for Adaptive Card version 1.3
Code changes commit for webexpythonsdk's Adaptive Card 1.1 to 1.3 version upgrade
@Sakthivel-Ramasamy
Copy link
Author

set_if_not_none() function from utils.py has been removed as I did not find any usage in the entire SDK!

@Joezanini Joezanini self-assigned this Sep 5, 2024
Copy link
Collaborator

@Joezanini Joezanini left a comment

Choose a reason for hiding this comment

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

The code was formatted well and passed the linter test. but when I went to test I ran into a Syntax Error.

The output indicated that the test run failed due to a SyntaxError in the file WebexPythonSDK/src/webexpythonsdk/models/cards/utils.py at line 58. The specific error message is SyntaxError: '(' was never closed, which means there is an unmatched opening parenthesis ( in the code.

To fix this, you need to locate the unmatched parenthesis in utils.py and ensure that every opening parenthesis has a corresponding closing parenthesis ).

@Sakthivel-Ramasamy
Copy link
Author

It is surprising. I tried the same and simulated an error to reach that particular piece of code, but it worked correctly. Can you please share the adaptive card code that you've used that led to this error?

@Joezanini
Copy link
Collaborator

Now I am getting a SyntaxError: f-string: expecting '}'. This is for src/webexpythonsdk/models/cards/utils.py Line 61. I think it has to do with the quotation mark placement.

@Joezanini Joezanini merged commit 4a556eb into WebexCommunity:main Oct 28, 2024
@Sakthivel-Ramasamy
Copy link
Author

Sakthivel-Ramasamy commented Nov 5, 2024

Code commit for #242

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