Skip to content

Checklist #4

@ZitaNemeckova

Description

@ZitaNemeckova

Checklist:

  • no extra files
    • Delete controller file when you move it.
    • No extra files like backup or any config files.
  • convert new and edit (if it applies to you)
    • if you changed only edit or new you should convert the other one too
  • use bindings not ManageIQ.angular.app.value('something', @something);
    • change fromManageIQ.angular.app.value('something', @something); to %some-component{'something' => @something} but use names from your files
      See here
  • controller is in component file
    • see this commit to see how to do it. Do not copy anything from there.
  • network.rb isn't changed
    • any change to network.rb is wrong. If you don't know how to remove it or you have more changes in commit that adds it make new commit where you only remove the changes. Or rebase.
  • only one new file in views/static/
  • no Ruby variables in views/static/
    • anything with @ or local Ruby variable should be passed via bindings, controller_name as well
  • using repository anywhere is big no-no.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions