Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Code refactor #17

Merged
merged 20 commits into from Jan 3, 2019
Merged

Code refactor #17

merged 20 commits into from Jan 3, 2019

Conversation

denvash
Copy link
Contributor

@denvash denvash commented Jan 3, 2019

Code Refactoring

  • re-arrange
  • add eslint
  • migrate to style-components
  • migrate to sass
  • Components - Containers - Dumb
  • No Errors in runtime
  • ExecuteButton fix

Table check list refactor

  • AlgorithmTable
  • DebugTable
  • DriverTable
  • StoredPipelinesTable
  • WorkerTable

Questions

  • sider height
  • ant-table-pagination
  • addPipe props in algorithm table
  • socket middlware: trying to register topic PROGRESS twice

Nice to have

  • Better eslint config

This change is Reviewable

@denvash denvash self-assigned this Jan 3, 2019
@denvash denvash requested a review from maty21 January 3, 2019 16:55
@denvash denvash merged commit 24e3920 into master Jan 3, 2019
hkube-ci pushed a commit that referenced this pull request Jan 3, 2019
* [+] Add eslint and update npm dependencies

* [+] Styled Component refactor start

* [~] Header as styled-comp

* [+] migration to Style-Component (1/2)

* [+] withState/Stateless components

* [~] Minor

* [~] cant npm start

* [~] fixing

* [+] containers - dumb

* [+] containers - dumb

* [+] containers - dumb

* [-] remove execute button

* [+] No errors, migrate to scss

* [+] add sass

* change from the ugly play icon with a circle around to something much prettier but with an incorrect name

* [+] Fix execute button, ready for merge

* [+] Fix all errors

* [~] Typo fix
 .... bump version [skip ci]
@denvash denvash deleted the code-refactor branch January 9, 2019 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants