Project

General

Profile

Actions

Refactor #22473

open

Refactor the react app folder structure to group files by domain

Added by Walden Raines about 6 years ago. Updated almost 6 years ago.

Status:
New
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
-
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

See https://community.theforeman.org/t/suggestion-refactor-the-react-app-folder-structure/7843/3

```
pp/
Header.js
Sidebar.js
App.js
reducers.js
routes.js
command/
Command.js
CommandContainer.js
CommandActions.js
CommandList.js
CommandItem.js
CommandHelper.js
commandReducer.js
product/
Product.js
ProductContainer.js
ProductActions.js
ProductList.js
ProductItem.js
ProductImage.js
productReducer.js
user/
User.js
UserContainer.js
UserActions.js
UserProfile.js
UserAvatar.js
userReducer.js
```


Related issues 2 (0 open2 closed)

Related to Foreman - Refactor #22683: Refactor the PasswordStrength folder structure to group files by domainClosedAvi Sharvit02/25/2018Actions
Related to Foreman - Refactor #25864: Refactor the FactChart component structure to group files by domainClosedMaria AgaphontzevActions
Actions #1

Updated by Walden Raines about 6 years ago

Sorry about the formatting in the summary, forgot this wasn't md!

app/
    Header.js
    Sidebar.js
    App.js
    reducers.js
    routes.js
command/
    Command.js
    CommandContainer.js
    CommandActions.js
    CommandList.js
    CommandItem.js
    CommandHelper.js
    commandReducer.js
product/
    Product.js
    ProductContainer.js
    ProductActions.js
    ProductList.js
    ProductItem.js
    ProductImage.js
    productReducer.js
user/
    User.js
    UserContainer.js
    UserActions.js
    UserProfile.js
    UserAvatar.js
    userReducer.js
Actions #2

Updated by Walden Raines about 6 years ago

  • Copied to Refactor #22474: Refactor the react app folder structure to group files by domain added
Actions #3

Updated by Walden Raines about 6 years ago

  • Copied to deleted (Refactor #22474: Refactor the react app folder structure to group files by domain)
Actions #4

Updated by The Foreman Bot about 6 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/5240 added
Actions #5

Updated by Tomáš Strachota almost 6 years ago

  • Related to Refactor #22683: Refactor the PasswordStrength folder structure to group files by domain added
Actions #6

Updated by Marek Hulán almost 6 years ago

  • translation missing: en.field_release set to 353
Actions #7

Updated by Walden Raines almost 6 years ago

  • Target version deleted (219)
Actions #8

Updated by Ohad Levy almost 6 years ago

  • Target version deleted (1.19.0)
  • Triaged set to No
  • Pull request deleted (https://github.com/theforeman/foreman/pull/5240)

seems like the linked PR was solving #22683 and not this one. is this a generic issue to refactor the other components to look like the Password Strength component ?

Actions #9

Updated by Ohad Levy almost 6 years ago

  • Status changed from Ready For Testing to New
Actions #10

Updated by Maria Agaphontzev over 5 years ago

  • Related to Refactor #25864: Refactor the FactChart component structure to group files by domain added
Actions

Also available in: Atom PDF