Skip to content

New settings UI clean up non regression #916

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

New settings UI clean up non regression #916

wants to merge 3 commits into from

Conversation

Migelangel04
Copy link
Contributor

@Migelangel04 Migelangel04 commented Sep 4, 2025 •
edited
Loading

What Is This Change?

  • Clean up the Settings Experiment which leaves the new settings page :)
  • Removed the old settings page and related files from the codebase which keeps are codebase clean

How Has This Been Tested?

  • Manual Testing

Basic checks:

  • npm run lint
  • npm run test

Advanced checks:

  • If Atlassian employee & Bitbucket changes: did you test with DC in mind? See Instructions

Recommendations:

  • Update the CHANGELOG if making a user facing change

Note: The e2e tests may need to be changed because of these changes

@Migelangel04 Migelangel04 marked this pull request as draft September 4, 2025 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bwieger-atlassian-com bwieger-atlassian-com Awaiting requested review from bwieger-atlassian-com bwieger-atlassian-com is a code owner

@sdzh-atlassian sdzh-atlassian Awaiting requested review from sdzh-atlassian sdzh-atlassian is a code owner

@cabella-dot cabella-dot Awaiting requested review from cabella-dot cabella-dot is a code owner

@marcomura marcomura Awaiting requested review from marcomura marcomura is a code owner

@akalia-atlassian akalia-atlassian Awaiting requested review from akalia-atlassian

@pastelsky pastelsky Awaiting requested review from pastelsky

@bhorai-atl bhorai-atl Awaiting requested review from bhorai-atl bhorai-atl is a code owner

@om-ukr om-ukr Awaiting requested review from om-ukr om-ukr is a code owner

@Blastoplex Blastoplex Awaiting requested review from Blastoplex Blastoplex will be requested when the pull request is marked ready for review Blastoplex is a code owner

@matt-lassian matt-lassian Awaiting requested review from matt-lassian matt-lassian will be requested when the pull request is marked ready for review matt-lassian is a code owner

@dchiew-atl dchiew-atl Awaiting requested review from dchiew-atl dchiew-atl will be requested when the pull request is marked ready for review dchiew-atl is a code owner

@amarg-at amarg-at Awaiting requested review from amarg-at amarg-at will be requested when the pull request is marked ready for review amarg-at is a code owner

@ccallcottstevens ccallcottstevens Awaiting requested review from ccallcottstevens ccallcottstevens will be requested when the pull request is marked ready for review ccallcottstevens is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Migelangel04