Skip to content

Conversation

@crowecawcaw
Copy link
Contributor

@crowecawcaw crowecawcaw commented Dec 11, 2025

What was the problem/requirement? (What/Why)

When using deadline bundle gui-submit --browse, there's an awkward button under job specific settings to swap job bundles. It looks weird.

The option was added a couple years ago in this PR: #35

Screenshot 2025-12-11 at 10 41 57 AM

What was the solution? (How)

Move the button to the bottom of the window next to the "Export bundle" button.

Screenshot 2025-12-11 at 12 51 39 PM

What is the impact of this change?

Cleaner bundle submitter.

How was this change tested?

Ran the command, saw the button was moved. Added unit tets

Was this change documented?

n/a

Does this PR introduce new dependencies?

No

Is this a breaking change?

No

Does this change impact security?

No


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@crowecawcaw crowecawcaw requested a review from a team as a code owner December 11, 2025 18:47
@github-actions github-actions bot added the waiting-on-maintainers Waiting on the maintainers to review. label Dec 11, 2025
@crowecawcaw crowecawcaw force-pushed the bundle branch 3 times, most recently from f4c91ea to 872a41d Compare December 11, 2025 22:36
@crowecawcaw
Copy link
Contributor Author

API changes not actually API changes and safe to ignore. No one should be depending on these imports even if they were not marked as internally only.

@crowecawcaw crowecawcaw changed the title fix: remove awkward bundle loading button when using gui-submit --browse fix: move bundle loading button when using gui-submit --browse Dec 11, 2025
Copy link
Contributor

@leongdl leongdl left a comment

Choose a reason for hiding this comment

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

API change was detected, but I don't think it applies in this UI situation.

@moorec-aws moorec-aws removed the waiting-on-maintainers Waiting on the maintainers to review. label Dec 29, 2025
@sonarqubecloud
Copy link

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.

3 participants