Skip to content

Edit Repository UI #334

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
Merged
merged 11 commits into from
Jul 28, 2025
Merged

Edit Repository UI #334

merged 11 commits into from
Jul 28, 2025

Conversation

gerry004
Copy link

@gerry004 gerry004 commented Jul 18, 2025 •
edited
Loading

This PR:

  • allows users to update repositories via the RepositoryInputModal
  • follows the same pattern as JobInputModal, handling state using editingRepository
  • updates the backend endpoint so that if the repo to be updated has the same name as the other, it doesn't throw an error
  • when editing repositories, the source locale dropdown is disabled because there should never be a case where the source locale changes after repo creation
  • adds unit tests for Repository Input Helpers

Quick video demo:
https://github.com/user-attachments/assets/c192cee7-0198-44fe-92d2-096ad68a0b0a

@gerry004 gerry004 requested a review from a team as a code owner July 18, 2025 15:30
byronantak
byronantak reviewed Jul 21, 2025
View reviewed changes
@gerry004 gerry004 requested a review from byronantak July 24, 2025 10:57
@gerry004 gerry004 force-pushed the gerryyang/edit-repository-ui branch from 1ec8f97 to a72fe11 Compare July 25, 2025 14:21
byronantak
byronantak approved these changes Jul 28, 2025
View reviewed changes
@gerry004 gerry004 enabled auto-merge (squash) July 28, 2025 10:42
@gerry004 gerry004 merged commit 4d966dd into master Jul 28, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@byronantak byronantak byronantak approved these changes

@mattwilshire mattwilshire Awaiting requested review from mattwilshire

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gerry004 @byronantak