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

Cause API #267

Closed
TBlueF opened this issue Jun 11, 2015 · 6 comments
Closed

Cause API #267

TBlueF opened this issue Jun 11, 2015 · 6 comments
Labels
help wanted We would appreciate contributions
Milestone

Comments

@TBlueF
Copy link

TBlueF commented Jun 11, 2015

An explanation of the Cause API and a list with all events and the causes they could have would be very great! Because i'm always very unshure what causes i can expect on an event.

@Deamon5550
Copy link
Contributor

Completion of this should wait until the scope and shape of SpongePowered/SpongeAPI#712 is fully defined.

@RobertHerhold RobertHerhold added the API not ready The API/code is not yet ready so we cannot merge this in yet label Jun 11, 2015
@Inscrutable Inscrutable added this to the Finish /plugin Pages milestone Jun 13, 2015
@Inscrutable
Copy link
Member

The Event-Refactoring branch ate that PR, so when that branch merges, this can happen.

@Inscrutable Inscrutable added help wanted We would appreciate contributions and removed API not ready The API/code is not yet ready so we cannot merge this in yet labels Sep 12, 2015
@Inscrutable
Copy link
Member

Some Lambda examples from Gabizou: https://gist.github.com/gabizou/915649871fadb235d0f0
Of particular importance, "plugins shouldn't be throwing empty causes".

@Inscrutable
Copy link
Member

Here's a fresh video from Gabizou going into some more details. https://youtu.be/4WhVqIM8Zac

@Tzky Tzky modified the milestones: v3.1.0, v3.0.0, v4.0.0 Mar 9, 2016
@Inscrutable Inscrutable modified the milestones: v4.0.0, v4.1.0 Jan 8, 2018
@Inscrutable
Copy link
Member

Do we now have enough documentation on causes to justify closing this issue?

@Inscrutable
Copy link
Member

This issue is being closed in favour of #469

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted We would appreciate contributions
Projects
None yet
Development

No branches or pull requests

5 participants