Skip to content
This repository was archived by the owner on Apr 14, 2026. It is now read-only.

Feature/pbs 25 05#2540

Merged
brianjgeiger merged 55 commits into
developfrom
feature/pbs-25-05
Apr 7, 2025
Merged

Feature/pbs 25 05#2540
brianjgeiger merged 55 commits into
developfrom
feature/pbs-25-05

Conversation

@brianjgeiger
Copy link
Copy Markdown
Contributor

Purpose

Merge the feature branch pbs-25-05 to develop.

bp-cos and others added 25 commits March 12, 2025 13:15
…t the date exists in the format. I also commented on some logic I am unable to produce
## Purpose
Fix a caching issue related to removing a supplement

## Summary of Changes
Added a try/catch
removed the node

## Side Effects
Should prevent future issues

https://openscience.atlassian.net/browse/ENG-5326
-   Ticket: [ENG-5027]

## Purpose
- Show deleted user's names in contributor fields

## Summary of Changes
- Add logic in osf-serializer to finagle deleted user data onto contributor model
- Update contributor-list/contributor component to look for deleted user data in case ContributorModel.user relationship is undefined

## Side Effects
- _may_ be introducing a new error when setting head tags, specifically these lines where we try to sparse-load contributors and their users https://github.com/CenterForOpenScience/ember-osf-web/blob/master/app/preprints/detail/route.ts#L155-L158

## QA Notes
- Verify that deleted users (probably via the admin app GDPR delete them?) have their names shown in preprint detail page

[ENG-5027]: https://openscience.atlassian.net/browse/ENG-5027
… to resubmit (#2528)

-   Ticket: https://openscience.atlassian.net/browse/ENG-7472

## Purpose
Make it possible to resubmit rejected preprint versions
* Send metrics for unverified DOIs

* fix(dcut): avoid resolving preprint guids

when `itemDoi` is set to an empty string, do not try finding a DOI for
that item by more convoluted means -- let the empty be.

* fix: avoid error tracking preprint download

---------

Co-authored-by: abram axel booth <boothaa@gmail.com>
…#2534)

## Purpose
- Prevent bug that prevents a write-user from updating registration metadata

## Summary of Changes
- Remove subjects relationship data from PUT request payload
  - Subjects are only editable by registration admins, so having this included in the request payload was preventing write users from editing metadata

https://openscience.atlassian.net/browse/ENG-7712
## Purpose
- Fix issues with write-users not being able to update registration metadata due to subjects being added to request payloads

## Summary of Changes
- Undo changes made in this PR #2534
- Update the node metadata form to show subject picker only for admin users
- Only show selected subjects for write and read users

https://openscience.atlassian.net/browse/ENG-7712
-   Ticket: [ENG-6927]

## Purpose
Issues with css. I thought the default "display: flex" was column???

## Summary of Changes
Add "flex-direction: column" to the main wrapper and removed it from the mobile class.
…nt selected folder (#2536)

-   Ticket: [ENG-7714] [ENG-7025]
-   Feature flag: n/a

## Purpose
- Fix issue where Configure Addon page's Save button was enabled when users had not yet selected a root folder
- Improve UX so users will see their currently selected root folder at the top of the page and root folder will be selected when navigating back to a view that shows the current root folder selection

## Summary of Changes
- Update logic for when Save button is disabled (`selectFolder` was an action, `selectedFolder` is the correct variable to use 😮‍💨 )
- Update confusing `hasRootFolder` property name to `requiresRootFolder`
- Add a "Selected folder:" display to help users know which folder is the currently selected root
- Add logic to pre-select radio button if the shown item is the currently selected root folder

[ENG-7714]: https://openscience.atlassian.net/browse/ENG-7714
[ENG-7025]: https://openscience.atlassian.net/browse/ENG-7025
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 7, 2025

Pull Request Test Coverage Report for Build 14317749780

Details

  • 527 of 550 (95.82%) changed or added relevant lines in 14 files are covered.
  • 5 unchanged lines in 3 files lost coverage.
  • Overall coverage increased (+1.0%) to 67.593%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/osf-components/addon/components/activity-log/component.ts 0 1 0.0%
lib/registries/addon/overview/recent-activity/route.ts 0 1 0.0%
app/services/analytics.ts 0 2 0.0%
lib/osf-components/addon/components/addons-service/configured-addon-edit/component.ts 10 12 83.33%
lib/osf-components/addon/components/activity-log/-components/activity-log-display/component.ts 130 135 96.3%
app/preprints/-components/submit/supplements/component.ts 0 6 0.0%
app/serializers/osf-serializer.ts 0 6 0.0%
Files with Coverage Reduction New Missed Lines %
lib/osf-components/addon/components/addons-service/configured-addon-edit/component.ts 1 67.69%
lib/osf-components/addon/components/subjects/manager/component.ts 1 70.1%
lib/osf-components/addon/components/subjects/browse/browse-manager/component.ts 3 52.94%
Totals Coverage Status
Change from base Build 14136307341: 1.0%
Covered Lines: 8468
Relevant Lines: 12047

💛 - Coveralls

@bp-cos bp-cos self-requested a review April 7, 2025 21:10
@brianjgeiger brianjgeiger merged commit 6cadff0 into develop Apr 7, 2025
10 checks passed
@brianjgeiger brianjgeiger deleted the feature/pbs-25-05 branch April 7, 2025 21:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants