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

Update EMM references to new name #23

Closed
mjangda opened this issue Aug 15, 2013 · 6 comments
Closed

Update EMM references to new name #23

mjangda opened this issue Aug 15, 2013 · 6 comments
Assignees

Comments

@mjangda
Copy link
Member

mjangda commented Aug 15, 2013

Before publicly releasing, we should try and update all references to extended-media-manager or emm to media-explorer or mexp_ to match the new name.

@ghost ghost assigned pepegar Aug 16, 2013
@pepegar
Copy link
Contributor

pepegar commented Aug 16, 2013

@mjangda, I've changed the naming of classes and functions containing EMM or extended media manager in f446167.

extended-media-manager --> media-explorer
emm --> me

Let me know if you like the names I've chose for them so I can submit a PR and merge.

@johnbillion
Copy link
Contributor

I wonder if mexp might be a better prefix than me?

@johnbillion
Copy link
Contributor

The textdomains will also need to change from emm.

@simonwheatley
Copy link
Contributor

I wonder if mexp might be a better prefix than me?

It's more likely to be unique, I'd guess; me is a very tempting prefix. :)

@pepegar
Copy link
Contributor

pepegar commented Aug 16, 2013

Yes, totally agree @johnbillion @simonwheatley, I'm changing it. Thanks for the feedback!

@pepegar
Copy link
Contributor

pepegar commented Aug 17, 2013

Pushed some more changes in fe424e8 following the advices of @simonwheatley and @johnbillion.

pepegar added a commit that referenced this issue Aug 19, 2013
Deleted references to EMM or Extended Media Manager. fixes #23.
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

No branches or pull requests

4 participants