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

[WIP] This PR includes the lib-retroshare changes for the notify feature of wire #133

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

PYRET1C
Copy link
Contributor

@PYRET1C PYRET1C commented Jan 7, 2024

There are some issues with the notify feature of wire which can be seen below on this PR:
RetroShare/RetroShare#2826

@PYRET1C
Copy link
Contributor Author

PYRET1C commented Jan 7, 2024

@defnax Please review this also

@defnax
Copy link
Contributor

defnax commented Jan 27, 2024

#ifdef RS_USE_WIRE
const uint32_t RS_FEED_TYPE_WIRE  = 0x8000;
#endif

@PYRET1C PYRET1C changed the title This PR includes the lib-retroshare changes for the notify feature of wire [WIP] This PR includes the lib-retroshare changes for the notify feature of wire Mar 17, 2024
@defnax
Copy link
Contributor

defnax commented Mar 18, 2024

can you update your code to use late libretroshare? we are then testing with very outdated code then wire.

@defnax
Copy link
Contributor

defnax commented Aug 25, 2024

@csoler needs to be checked

@defnax
Copy link
Contributor

defnax commented Aug 25, 2024

can you merge this when its ok @csoler

@PYRET1C
Copy link
Contributor Author

PYRET1C commented Aug 26, 2024

it still needs improvements. i will do them when i get time. I am sorry, i have been too busy past few months.

@defnax
Copy link
Contributor

defnax commented Aug 26, 2024

this year?

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