Skip to content
This repository has been archived by the owner on Jul 27, 2019. It is now read-only.

Better name for publish method #16

Closed
lukaszb opened this issue Feb 20, 2017 · 0 comments
Closed

Better name for publish method #16

lukaszb opened this issue Feb 20, 2017 · 0 comments

Comments

@lukaszb
Copy link
Contributor

lukaszb commented Feb 20, 2017

Currently we are using publish name for storing events as it is used directly from eventsourcing library. This might be misleading however - it's actually storing an event. We should call it simply store_event instead.

Not sure if we want to keep old method. djangoevents is not mature yet so we can change API without deprecating it first. OTOH, we might want to inform all users by issuing deprecationwarning (https://docs.python.org/3/library/warnings.html#default-warning-filters).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant