Skip to content

add assistant#555

Merged
erdgeist merged 2 commits intofrab:masterfrom
elad-eyal:add_assistant
Oct 28, 2019
Merged

add assistant#555
erdgeist merged 2 commits intofrab:masterfrom
elad-eyal:add_assistant

Conversation

@elad-eyal
Copy link
Collaborator

A new user role in an event: "assistant" . Assistant has the same
permissions as a speaker, but their availability does not count.
The assistant is notified by mail of changes to the event, similar to
presenter.

Prospective assistants can join events by use of the join token,
similar to co-speakers.

see #488

@elad-eyal elad-eyal force-pushed the add_assistant branch 2 times, most recently from 44f698c to c93d8a3 Compare September 25, 2019 08:53
@elad-eyal elad-eyal mentioned this pull request Oct 4, 2019
@elad-eyal elad-eyal force-pushed the add_assistant branch 2 times, most recently from e748c67 to 09ce23f Compare October 8, 2019 21:22
@elad-eyal elad-eyal force-pushed the add_assistant branch 2 times, most recently from 4c882f1 to 31c8afc Compare October 25, 2019 13:14
@erdgeist
Copy link
Contributor

Basically, the PR looks good to me, but I would want @manno 's opinion as well. Also, I strongly dislike the term "stakeholder" for multiple reasons. Maybe ON_SITE or ON_VENUE would be a more neutral descriptor for that group?

@elad-eyal
Copy link
Collaborator Author

elad-eyal commented Oct 25, 2019

Thanks erdgeist.

Re ON SITE: they're not necessarily on site. I see the assistant role used in two ways: (1) the speaker has a secretary which needs to get all relevant emails and is in charge of the speaker's travel arrangements and so forth. (2) a mentor capacity, someone who helps the speakers make sure their slides are attractive, well written, etc. She needs access to all files, and needs to know if the event was moved/rescheduled etc.

I am adding a commit adding i18n to the settings -> notification page (it was part English before), and also removed the usage of 'stakeholder' externally. Instead I use "people". So 'stakeholder' is used in code only.

image

image

image

@erdgeist
Copy link
Contributor

erdgeist commented Oct 26, 2019

I wonder, if the term "subscriber" properly transports the meaning. If you look at why we build this list in the first place, is to ensure everyone who needs to be notified about state changes of the event … actually is notified. This is what a subscription is. So I think, this term might be used instead of steakholder ;)

In other contexts the term "watchers" is used, like in ticket systems.

@elad-eyal
Copy link
Collaborator Author

I thought "watchers" to be confusing is the sense that people come to a conference in order to watch the actual event.

So I changed all "stakeholders" to "subscribers" - internally the term was not used.

I kept the external wording "send to 2 people" and did not use "2 subscribers" becuase I think it's clearer.

A new user role in an event: "assistant" . Assistant has the same
permissions as a speaker, but their availability does not count.
The assistant is notified by mail of changes to the event, similar to
presenter.

Prospective assistants can join events by use of the join token,
similar to co-speakers.

The group of people who needs to be notified by e-mail when something
is changed with the event is now called "subscribers".

see frab#488
# Conflicts:
#	app/views/conferences/_notification_section.html.haml
@erdgeist erdgeist merged commit c54c98b into frab:master Oct 28, 2019
@elad-eyal elad-eyal deleted the add_assistant branch October 28, 2019 19:10
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.

2 participants