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

QXmppAccountMigrationManager: Make public the extensions getter/setter #639

Merged
merged 2 commits into from
Aug 18, 2024

Conversation

pasnox
Copy link
Contributor

@pasnox pasnox commented Jun 29, 2024

This is needed because without that change applications can not store and use app specific data. By example Kaidan export and import its roster settings which requires that all migration tasks finished before importing roster related data.

PR check list:

  • Document your code
  • Add \since QXmpp 1.X, QXMPP_EXPORT
  • Update doc/doap.xml
  • Add unit tests
  • Format the code: Run clang-format -i src/<edited-file(s)> tests/<edited-file(s)>

@pasnox pasnox force-pushed the master-migrationmanager branch 3 times, most recently from 57f0679 to 8b984e3 Compare July 30, 2024 15:10
This is needed because without that change applications can not store
and use app specific data. By example Kaidan export and import its
roster settings which requires that all migration tasks finished before
importing roster related data.
@lnjX
Copy link
Member

lnjX commented Aug 18, 2024

Sorry for the delay. I just noticed QXmppExportData is missing any documentation. Would you like to fix that?

@lnjX lnjX merged commit ee72d41 into qxmpp-project:master Aug 18, 2024
10 of 15 checks passed
@pasnox pasnox deleted the master-migrationmanager branch August 24, 2024 14:59
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