Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: edit project design 🧑‍🎨 #1346

Merged
merged 2 commits into from
Mar 5, 2025

Conversation

luke-mcfarlane-rocketlab
Copy link
Contributor

@luke-mcfarlane-rocketlab luke-mcfarlane-rocketlab commented Feb 27, 2025

JIRA Ticket

BSS-667

Description

Added the ability to edit a project design.

Proposed Changes

  • added edit project dialog
  • added update project form

How to Test

  1. Open a project
  2. Navigate to the "Actions" tab
  3. Click "Edit Project" button
  4. Follow the instructions
  5. Verify the project has updated (easiest way is likely by downloading the new project file in the edit project dialog)

Additional Information

Checklist

  • I have confirmed all commits have been signed.
  • I have added JSDoc style comments to any new functions or classes.
  • Relevant documentation such as READMEs, guides, and class comments are updated.

Signed-off-by: Luke McFarlane <luke.mcfarlane@rocketlab.com.au>
Signed-off-by: Luke McFarlane <luke.mcfarlane@rocketlab.com.au>
Copy link
Contributor

@ranisa-gupta16 ranisa-gupta16 left a comment

Choose a reason for hiding this comment

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

Looks good tm. I am able to edit and upload that file in here an designer
Just a thought that we might need to show something visually to the user that the file has been uploaded.

@luke-mcfarlane-rocketlab luke-mcfarlane-rocketlab merged commit a98402b into main Mar 5, 2025
2 checks passed
@luke-mcfarlane-rocketlab luke-mcfarlane-rocketlab deleted the feat/edit-survey-design branch March 5, 2025 03:25
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants