Skip to content

Don't overwrite shell when updating user entry #889

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

Merged
merged 2 commits into from
Apr 25, 2025

Conversation

adombeck
Copy link
Contributor

@adombeck adombeck commented Apr 17, 2025

The user's shell should be configurable locally and not overwritten when the user info is being refreshed.

Closes #880
UDENG-6659

@codecov-commenter
Copy link

codecov-commenter commented Apr 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.44%. Comparing base (c24afd9) to head (5226dc9).
Report is 383 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #889   +/-   ##
=======================================
  Coverage   85.43%   85.44%           
=======================================
  Files          80       80           
  Lines        5547     5550    +3     
  Branches      109      109           
=======================================
+ Hits         4739     4742    +3     
  Misses        753      753           
  Partials       55       55           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@adombeck adombeck force-pushed the 880-do-not-update-shell-and-home branch 2 times, most recently from 381f51a to 5cbba79 Compare April 17, 2025 15:42
@adombeck adombeck changed the title Don't overwrite home and shell when updating user entry Don't overwrite shell when updating user entry Apr 17, 2025
@adombeck adombeck marked this pull request as ready for review April 17, 2025 15:58
@adombeck adombeck requested a review from a team as a code owner April 17, 2025 15:58
@3v1n0 3v1n0 linked an issue Apr 21, 2025 that may be closed by this pull request
2 tasks
The shell should be configurable locally and not overwritten when the
user info is refreshed.
@adombeck adombeck force-pushed the 880-do-not-update-shell-and-home branch from 5cbba79 to 5226dc9 Compare April 22, 2025 20:20
@adombeck adombeck requested a review from 3v1n0 April 25, 2025 09:49
@adombeck
Copy link
Contributor Author

I would like to get this merged in time for the release.

@adombeck adombeck merged commit 424966b into main Apr 25, 2025
14 of 16 checks passed
@adombeck adombeck deleted the 880-do-not-update-shell-and-home branch April 25, 2025 13:28
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.

Issue: Cannot update shell
3 participants