-
Notifications
You must be signed in to change notification settings - Fork 17
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
[NPD][IN-477][Reviews]Make sure Reviews work with post-NPD changes #153
[NPD][IN-477][Reviews]Make sure Reviews work with post-NPD changes #153
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for addressing these reviews changes @adlius!
Just a few things that could be removed, and then where we were checking to see if a node was public, we should check the preprint instead.
import { ArrayPromiseProxy, loadRelation } from 'ember-osf/utils/load-relationship'; | ||
|
||
const PAGE_SIZE = 6; | ||
|
||
|
||
export default Component.extend({ | ||
pageNumber: 0, | ||
selectedFile: null, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can probably be removed
import { ArrayPromiseProxy, loadRelation } from 'ember-osf/utils/load-relationship'; | ||
|
||
const PAGE_SIZE = 6; | ||
|
||
|
||
export default Component.extend({ | ||
pageNumber: 0, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can probably be removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me @adlius!
@adlius, @jamescdavis, checking in on this one. Can we get these reviews changes on staging3? I assume this needs to go out with NPD. |
@brianjgeiger, can this go on staging3? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
thank you @jamescdavis! |
Purpose
To maintain functional parity with
ember-osf-preprints
once NPD changes are in effect.Summary of Changes/Side Effects
Deleted some tests that are no longer valid. Change the detail page so that it shows the same header as the post-NPD preprint detail page.
Ticket
https://openscience.atlassian.net/browse/IN-477
Reviewer Checklist
CHANGELOG.md