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

Change drupal login to (ixp-agnostic) portable cms login #59

Merged
merged 2 commits into from
Sep 12, 2013

Conversation

rowanthorpe
Copy link
Contributor

This is optimistically done in case someone (me?) gets enthusiastic enough to add (configurable) handling for other CMSes, in addition to Drupal. Either way the more generic wording can't hurt. Also, perhaps you will prefer to find another wording than "superuser" but I guess either way you prefer to de-INEXify the text (let me know if you want me to tweak that bit differently)...

@barryo
Copy link
Member

barryo commented Sep 12, 2013

No general issues with this - except it'll break on our end :( - we already have files that link to auth/drupal.

Let's park it until it's more than an aspiration on your end :P

@barryo barryo closed this Sep 12, 2013
@barryo
Copy link
Member

barryo commented Sep 12, 2013

Or, if you want to dig out the reference to drupal in views/_skins/inex/staff-links.phtml and update to cms then I'll happily merge.

@barryo barryo reopened this Sep 12, 2013
@rowanthorpe
Copy link
Contributor Author

Hopefully that extra change is what you need? I don't exactly know how (smarty?) derives the syntax...

barryo added a commit that referenced this pull request Sep 12, 2013
Change drupal login to (ixp-agnostic) portable cms login
@barryo barryo merged commit 86692c9 into inex:master Sep 12, 2013
@barryo
Copy link
Member

barryo commented Sep 12, 2013

That's pretty much it. I'll tweak it a little, thanks.

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