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

with next upgrade override to_s method in User class #157

Closed
kdid opened this issue Mar 24, 2017 · 2 comments
Closed

with next upgrade override to_s method in User class #157

kdid opened this issue Mar 24, 2017 · 2 comments
Assignees

Comments

@kdid
Copy link
Contributor

kdid commented Mar 24, 2017

This issue serves as a reminder to update the to_s method on the user class with username instead of email. Sufia 7.3 fixes the "Works Created" link not working ( #145 ), but this change is additionally required for it to work.

Note that we have started a 7.3 upgrade branch
https://github.com/nulib/institutional-repository/tree/7.3-upgrade

but in case we don't end up using it (go directly to Hyrax, this will serve as a reminder to make this code change.

In app/models/usrs

replace

  def to_s
    email
  end

with

  def to_s
    username
  end
@davidschober
Copy link
Contributor

verify with #160

@bmquinn
Copy link
Contributor

bmquinn commented Mar 28, 2017

Fixed in #164

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants