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

Add new page: Event Management System #32

Merged
merged 2 commits into from
Feb 23, 2018
Merged

Conversation

jwflory
Copy link
Member

@jwflory jwflory commented Feb 22, 2018

This follows up from our discussion at last week's eboard meeting on Feb. 16. @axk4545 went through the process of requesting a room through the RIT Event Management System (EMS). He shared some of what he found and we took some quick notes to document in the Runbook. This is the outcome of that discussion.

@axk4545 and @Tjzabel, either of you would be appreciated since you were there for the discussion.

@Serubin, @ct-martin, or @TaylorBowling, your feedback is also welcome since you were not there for the discussion and don't have the background context.

The rendered page locally looks something like this:

RITlug Runbook: Event Management System

@jwflory jwflory added new page Introducing new documentation pages doc: Events Affects content in Events category labels Feb 22, 2018
@jwflory jwflory self-assigned this Feb 22, 2018
@jwflory jwflory requested a review from a team February 22, 2018 02:09
Copy link
Member

@axk4545 axk4545 left a comment

Choose a reason for hiding this comment

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

see comment


RIT EMS is used when you need to make a reservation for a room on RIT's campus.
Extra options are enabled on student accounts linked to a student organization.
For an error or booking error, RIT EMS is also used.
Copy link
Member

Choose a reason for hiding this comment

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

not sure what this means...

@jwflory
Copy link
Member Author

jwflory commented Feb 22, 2018

@axk4545 The latest commit omits that line. Even I'm not completely sure what was meant – my notes were confusing. Let me know if this makes more sense now.

@axk4545 axk4545 merged commit 7c4bc3d into master Feb 23, 2018
@axk4545 axk4545 deleted the add/event-management-system branch February 23, 2018 01:47
@jwflory
Copy link
Member Author

jwflory commented Feb 23, 2018

@axk4545 Thanks for the updated review. For future reference, let's make sure new pages or major revisions get +2 approved reviews by eboard members before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc: Events Affects content in Events category new page Introducing new documentation pages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants