Skip to content

Conversation

@sum12
Copy link

@sum12 sum12 commented May 7, 2021

this allow the CredComponent to be reused across the all other
components.
Since at any time only one using compoenent (here PullComponent only for
now) is visible only one these sharing entities will try to get hold of
the shared component via the RefCell

@sum12 sum12 mentioned this pull request May 7, 2021
@sum12 sum12 force-pushed the cred-component-via-refcell branch 3 times, most recently from 298f4c8 to aeea25a Compare May 9, 2021 15:42
@sum12
Copy link
Author

sum12 commented May 12, 2021

@extrawurst I know this is probably annoying. But can you give this another approval ?
I have done a make check make test and make build-release so atleast the linux part should pass the CI

(Thinking of another smaller PR just become a contributor before trying to work further on this on)

@sum12 sum12 marked this pull request as ready for review May 12, 2021 11:52
sum12 added 4 commits May 12, 2021 15:25
…th main app

this allow the CredComponent to be reused across the all other
components.
Since at any time only one using compoenent (here PullComponent only for
now) is visible only one these sharing entities will try to get hold of
the shared component via the RefCell
…th main app

This component is shared with pullcomponent, which was updated also
updated to use it via this shared instance.
this is shared object with PullComponent and PushComponent
@sum12 sum12 force-pushed the cred-component-via-refcell branch from aeea25a to 1bf96f7 Compare May 12, 2021 13:59
@sum12
Copy link
Author

sum12 commented May 18, 2021

@extrawurst any input on this ?

@sum12
Copy link
Author

sum12 commented May 27, 2021

[try 2] @extrawurst does this go in the right direction ?

@extrawurst
Copy link
Collaborator

[try 2] @extrawurst does this go in the right direction ?

sorry for not giving you feedback on this yet, I hope to be able to on the weekend

@sum12
Copy link
Author

sum12 commented Jul 20, 2021

Is this still relevant ( even after resolving the conflicts) ?
@extrawurst

@stale
Copy link

stale bot commented Mar 17, 2024

This issue has been automatically marked as stale because it has not had any activity half a year. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the dormant Marked by stale bot on close label Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dormant Marked by stale bot on close

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants