Skip to content

[#PLAT-696]Add preprints and registrations to user profile page#8234

Open
chennan47 wants to merge 12 commits intoCenterForOpenScience:developfrom
chennan47:feature/profile_page
Open

[#PLAT-696]Add preprints and registrations to user profile page#8234
chennan47 wants to merge 12 commits intoCenterForOpenScience:developfrom
chennan47:feature/profile_page

Conversation

@chennan47
Copy link
Copy Markdown
Contributor

@chennan47 chennan47 commented Mar 20, 2018

Purpose

Update user's profile page by adding public registrations and preprints. Remove public components.

Changes

Before change:
screen shot 2018-03-20 at 10 14 56 am
After change:
User with projects, registrations, preprints and quickfiles view his own profile page:
screen shot 2018-03-23 at 3 16 32 pm
Another user view a user's profile page who has projects, registrations, preprints and quickfiles
screen shot 2018-03-23 at 3 16 39 pm
User without any projects, registrations, preprints or quickfiles view his own profile page
screen shot 2018-03-23 at 3 21 38 pm
Another user view a user's profile page who has no projects, registrations, preprints or quickfiles
screen shot 2018-03-23 at 3 23 15 pm

QA Notes

  1. checkout user's profile page.
    things to check:
  • If user has no preprints, preprints widget will still show up but saying "The user has no preprints"

  • If user has no public registrations, registrations widget will still show up but saying "The user has no public registrations"

  • If user has no quickfiles, quickfiles widget will disappear

Side Effects

Ticket

https://openscience.atlassian.net/browse/PLAT-696

@Johnetordoff
Copy link
Copy Markdown
Contributor

Johnetordoff commented Mar 27, 2018

@chennan47 Looks good 👍 , but you have a single test failing, https://travis-ci.org/CenterForOpenScience/osf.io/jobs/357552537... There's also a typical elasticsearch fail, but this one is separate.

@Johnetordoff
Copy link
Copy Markdown
Contributor

@chennan47 You still have a test failure but it's appears to be out of date, and no longer part of your PR, so it really should pass. I'm passing this along to @sloria, because it seems like a travis problem.

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