Skip to content

Update stale - #77

Open
charlesRmajor wants to merge 7 commits into
istornz:mainfrom
repstudio:update-stale
Open

Update stale#77
charlesRmajor wants to merge 7 commits into
istornz:mainfrom
repstudio:update-stale

Conversation

@charlesRmajor

Copy link
Copy Markdown
Contributor

Update "update" function to match Apple's updated method, including supporting staleIn time same as in createActivity.

The previous method has been deprecated

NOTE: this new "update" function requires iOS version 16.2, not 16.1. I replaced all of the 16.1's with 16.2, but I don't know if that's what you'll want for the package. My need definitely doesn't cover supporting users who insist on being on 16.1.

(Thanks for making this easy for us, Apple, lol)

@istornz istornz left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM, can you please look at my two little comments/suggestions? 🥇

Comment thread lib/live_activities.dart Outdated
Comment thread lib/live_activities.dart Outdated
@istornz

istornz commented Apr 4, 2024

Copy link
Copy Markdown
Owner

@charlesRmajor Sorry for the delay, I was in vacation :/

It's good for me, can you fix conflicts before merging?

Thanks a lot!

}
}

Future<File> _compressToSize(

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What is the purpose of this compress ?

@EArminjon

Copy link
Copy Markdown
Contributor

Is this will be merged ?

@istornz

istornz commented Feb 12, 2025

Copy link
Copy Markdown
Owner

@EArminjon I hope but we need to fix conflicts before

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants