Project

General

Profile

Refactor #22473

Refactor the react app folder structure to group files by domain

Added by Walden Raines 10 months ago. Updated 4 months ago.

Status:
New
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
Team Backlog:
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

Related to Foreman - Refactor #22683: Refactor the PasswordStrength folder structure to group files by domainClosed2018-02-25

History

#1 Updated by Walden Raines 10 months 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

#2 Updated by Walden Raines 10 months ago

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

#3 Updated by Walden Raines 10 months ago

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

#4 Updated by The Foreman Bot 9 months ago

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

#5 Updated by Tomáš Strachota 6 months ago

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

#6 Updated by Marek Hulán 6 months ago

  • Legacy Backlogs Release (now unused) set to 353

#7 Updated by Walden Raines 5 months ago

  • Target version deleted (219)

#8 Updated by Ohad Levy 4 months ago

  • Triaged set to No
  • Target version deleted (1.19.0)
  • 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 ?

#9 Updated by Ohad Levy 4 months ago

  • Status changed from Ready For Testing to New

Also available in: Atom PDF