Skip to content

Integrate core lib #67

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

Merged
merged 8 commits into from
Jun 24, 2022
Merged

Integrate core lib #67

merged 8 commits into from
Jun 24, 2022

Conversation

nesium
Copy link
Contributor

@nesium nesium commented Jun 23, 2022

New features:

  • Roster is loaded dynamically
  • Messages can be sent and received (I've included the user's JID in the roster so that we can send messages to ourselves with another IM)

Known bugs:

  • Roster is only loaded after a restart after a successful login.
  • If the connection is lost, you're on your own.

Note: The XMPP model classes reside in ProseCoreTCA as I'm trying to figure out what goes where. ProseCore will ideally not have any dependencies other than ProseCoreClientFFI. Also I'm sticking to XMPP terminology where applicable.

@nesium nesium requested a review from RemiBardon June 23, 2022 15:06
@RemiBardon
Copy link
Member

I'll have a look fist thing in the morning tomorrow, my PR #68 took longer than expected…

Copy link
Member

@RemiBardon RemiBardon left a comment

Choose a reason for hiding this comment

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

Great work! Just a few comments to address and we can merge this PR 🥳

(Note: I haven't tested the project, should I?)

@RemiBardon RemiBardon added the feature New feature to implement label Jun 24, 2022
Copy link
Member

@RemiBardon RemiBardon left a comment

Choose a reason for hiding this comment

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

All good, thank you very much 🙏🏼

@nesium nesium merged commit 57fe473 into master Jun 24, 2022
@nesium nesium deleted the integrate-core-lib branch June 24, 2022 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature to implement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants