Skip to content

Update app.py #182

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update app.py #182

wants to merge 1 commit into from

Conversation

brianbterry
Copy link

Removed duplicated send_animated_message function body

@brianbterry
Copy link
Author

Should close issue #180

Removed duplicated function body in new location
@brianbterry brianbterry reopened this Mar 25, 2025
@willydouhard
Copy link
Contributor

I think you are removing the entire progress animation no?

@@ -33,30 +33,6 @@ async def send_animated_message(
except asyncio.CancelledError:
msg.content = base_msg
await msg.update() # Final static message
base_msg: str,

Choose a reason for hiding this comment

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

Looks like this was the older body of the function that got left behind

@brianterry-ama
Copy link

Sorry for the late reply. There is what I think was an old body of the function accidentally left in. You can see it if you expand up a couple times. This is causing #180 where the loader animation doesn't work out of the box. Might want the keep the body I deleted but I think this is the one desired.

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