Skip to content

[Rebase] [ENG-9177] Removed ember#11531

Open
antkryt wants to merge 6 commits intoCenterForOpenScience:feature/pbs-26-2from
antkryt:fix/ENG-9177
Open

[Rebase] [ENG-9177] Removed ember#11531
antkryt wants to merge 6 commits intoCenterForOpenScience:feature/pbs-26-2from
antkryt:fix/ENG-9177

Conversation

@antkryt
Copy link
Contributor

@antkryt antkryt commented Jan 6, 2026

Purpose

Original PR was rebased and contains unrelated commits.
This PR reconstructs the intended changes by cherry-picking the last 6 commits only.

Changes

QA Notes

Documentation

Side Effects

Ticket

https://openscience.atlassian.net/browse/ENG-9177

Copy link
Contributor

@Vlad0n20 Vlad0n20 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
There is one point with tests


def test_get_user_by_external_info(self):
service_url = 'http://localhost:5000/dashboard/'
service_url = 'http://localhost:5000/my_projects/'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test assertions use /my_projects/ but actual route is /myprojects/

@adlius adlius changed the base branch from feature/pbs-25-25 to feature/pbs-26-2 February 4, 2026 05:10
@adlius
Copy link
Collaborator

adlius commented Feb 4, 2026

@antkryt There are some merge conflicts that needs taken care of.

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.

4 participants