Skip to content

Conversation

@create-issue-branch
Copy link
Contributor

@create-issue-branch create-issue-branch bot commented Mar 4, 2022

When editing a project we need to support the following functionality:

  • New project components ("files") added to the project must be created
  • Removed project components must be destroyed
  • A project has a 'default' component (e.g. main.py) This can not be deleted or have its name or extension changed but it can have its content changed.
  • If the update process fails at any stage no changes should be persisted
    closes Bug fix: Allow saving of new files added to project #27

@raspberrypigithubbot
Copy link

@raspberrypigithubbot
Copy link

@raspberrypigithubbot
Copy link

@raspberrypigithubbot
Copy link

@raspberrypigithubbot
Copy link

@raspberrypigithubbot
Copy link

@ArayB ArayB marked this pull request as ready for review March 10, 2022 14:21
Copy link
Contributor

@loiswells97 loiswells97 left a comment

Choose a reason for hiding this comment

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

Just a few minor comments/questions from me:

  1. Should the errors say ''cannot" instead of "can not" in config/locales/en.yml?
  2. What does accepts_nested_attributes :components do in app/models/project.rb?

@raspberrypigithubbot
Copy link

Copy link
Contributor

@loiswells97 loiswells97 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 to me 👍

@ArayB ArayB merged commit af505fc into main Mar 10, 2022
@ArayB ArayB deleted the issues/27-Bug_fix_Allow_saving_of_new_files_added_to_project branch March 10, 2022 16: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.

Bug fix: Allow saving of new files added to project

4 participants