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

Actions queue #30

Closed
wants to merge 2 commits into from
Closed

Actions queue #30

wants to merge 2 commits into from

Conversation

ssestak
Copy link
Contributor

@ssestak ssestak commented Oct 25, 2024

Add ActionsQueue into FuturedHelpers.

Update ModalCoverModel to able presenting modal screens via ActionsQueue.

@ssestak ssestak requested a review from samoilyk October 25, 2024 10:57
actor ActionsCache<A: QueueAction> {
let subject: CurrentValueSubject<A?, Never> = CurrentValueSubject(nil)

private var actions = [A]() {
Copy link
Contributor

Choose a reason for hiding this comment

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

jinak mozna se da zbavit toho subjectu tim, ze se z toho udela @published property. Da se na to subscribovat, drzi to current value a bude to o par radku mensi

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Skúsil som to ale potom by musel byť ActionsQueue.init asynchrónny nie ?
A to kvôli týmto pár riadkom (viz screenshot).

image

Copy link
Contributor

Choose a reason for hiding this comment

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

mrkneme se na to spolu dneska 👍

@ssestak ssestak force-pushed the feature/ActionsQueue branch from 2791b77 to 9f8c432 Compare November 5, 2024 16:52
@ssestak ssestak closed this Nov 7, 2024
@ssestak ssestak deleted the feature/ActionsQueue branch November 7, 2024 14:52
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