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

added safe public methods to store #516

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

Conversation

artem-webdev
Copy link
Contributor

@artem-webdev artem-webdev commented Jan 28, 2021

added safe public methods to store

for exapmle

func (h *waHandler) HandleNewContact(contact whatsapp.Contact) {

	err := h.wac.Store.AddContact(contact)
	if err != nil {
		log.Println(err)
		return 
	}

}
...etc

@beshoo beshoo requested a review from SchulteMK January 31, 2021 00:17
@artem-webdev
Copy link
Contributor Author

it can be seen that the store does not use the connect, so let there be methods for store object

@artem-webdev
Copy link
Contributor Author

@SchulteMK Still in progress? - no, this is final

@beshoo beshoo requested review from Rhymen and SchulteMK and removed request for SchulteMK February 7, 2021 22:24
@beshoo
Copy link
Collaborator

beshoo commented Feb 7, 2021

@SchulteMK please can you review this PL.
With Regard

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

2 participants