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 get_event_state and get_logevent_state natives #821

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

Added get_event_state and get_logevent_state natives #821

wants to merge 12 commits into from

Conversation

SmileYzn
Copy link
Contributor

@SmileYzn SmileYzn commented Jan 7, 2020

I don't think I'll need to create variables in my plugin anymore to know if the event is active or inactive

😃 🎂

Added getForwardState
Added getForwardState
added get_event_state native
Added getForwardState
Added getForwardState
Updated getForwardState
 getForwardState
Added get_logevent_state
AddedForwardState enum for get_event_state and get_logevent_state
@SmileYzn
Copy link
Contributor Author

SmileYzn commented Jan 7, 2020

I forgot to add these natives to include files, someone can help?

Added get_event_state and get_logevent_state
Added MENU_KEY_ALL to avoid terrible 1023 constant
@SmileYzn
Copy link
Contributor Author

@Arkshine

I forgot something?

Added TIME_ELEMENT_PERMANENTLY to output.

Ps.
This is supposed to be used?
@SmileYzn
Copy link
Contributor Author

Set TIME_ELEMENT_PERMANENTLY to outupt of get_time_length

@Arkshine
Copy link
Member

Set TIME_ELEMENT_PERMANENTLY to outupt of get_time_length

How this change is related to this PR?

@SmileYzn
Copy link
Contributor Author

I do not know How i can open another :(

@asherkin
Copy link
Member

I do not know How i can open another :(

This is because you've created this PR off your master branch - you should have a read of the GitHub documentation on the PR process and how to use Git effectively.

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.

3 participants