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

Retain existing authState.creds.me properties and add lid #563

Merged
merged 1 commit into from
Jan 2, 2024

Conversation

kyraex
Copy link
Contributor

@kyraex kyraex commented Dec 28, 2023

Previously adding lid does not consider current property values of authState.creds.me which would lost on the next socket reconnect. This fix ensure current values are maintained

@whiskeysockets-bot
Copy link
Contributor

Thanks for your contribution.

The next step is to wait for review and approval to merge it to main repository

The community can help reacting with a thumb up (:thumbsup:) for approval and rocket (:rocket:) for who has tested it.

To test this PR you can run the following command below:

# NPM
npm install @whiskeysockets/baileys@kyraex/Baileys#retain-creds
# YARN v2
yarn add @whiskeysockets/baileys@kyraex/Baileys#retain-creds

Copy link
Collaborator

@PurpShell PurpShell left a comment

Choose a reason for hiding this comment

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

You are correct, my bad when adding this

Copy link
Contributor

@andresayac andresayac left a comment

Choose a reason for hiding this comment

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

I agree with the change

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.

None yet

5 participants