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

Refactorize visualization Capacity and Infrastructure #5

Conversation

nicolas-brousse
Copy link
Member

@nicolas-brousse nicolas-brousse commented Jun 19, 2024

  • Implement form
  • Implement turbo frame or stream
  • Re-Implement LeaderLine
  • Cleanup code view
  • Try to optimize SQL queries / add cache
  • Remove old code
  • Move some element in components?
  • Tests

@nicolas-brousse nicolas-brousse added the enhancement New feature or request label Jun 19, 2024
@nicolas-brousse nicolas-brousse self-assigned this Jun 19, 2024
@B-Rass B-Rass force-pushed the upgrade-to-bootstrap-4 branch 2 times, most recently from f511273 to 8925664 Compare June 28, 2024 15:33
@nicolas-brousse nicolas-brousse force-pushed the refactorize-visualization-capacity-and-infrastructure branch 2 times, most recently from 5d22712 to fadf037 Compare July 3, 2024 14:42
@nicolas-brousse nicolas-brousse force-pushed the refactorize-visualization-capacity-and-infrastructure branch from 3c07270 to 80b33f8 Compare July 10, 2024 14:51
@nicolas-brousse nicolas-brousse force-pushed the refactorize-visualization-capacity-and-infrastructure branch from 80b33f8 to 72f0be4 Compare July 10, 2024 14:54
@nicolas-brousse nicolas-brousse marked this pull request as ready for review August 7, 2024 14:23
@nicolas-brousse nicolas-brousse force-pushed the refactorize-visualization-capacity-and-infrastructure branch from bf8e7a5 to 3b9b1f8 Compare August 7, 2024 14:26
@nicolas-brousse nicolas-brousse force-pushed the refactorize-visualization-capacity-and-infrastructure branch from 3b9b1f8 to 33eb8eb Compare August 7, 2024 14:28
.where.not(network_types: [])
# .includes(:cards, :ports => [:connection => [:port, :cable =>[:connections => [:port => :card]]]]).
@concentrateurs_ids = [383, 384, 1043, 1044]
@concentrateurs = Server.where(id: @concentrateurs_ids).includes(:ports => :connection, :cards => [:ports => :connection])
Copy link

Choose a reason for hiding this comment

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

Don't we want to remove those =>

Copy link
Member Author

Choose a reason for hiding this comment

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

We could

app/helpers/servers_helper.rb Outdated Show resolved Hide resolved
Co-authored-by: B-Rass <B-Rass@users.noreply.github.com>
Copy link

Choose a reason for hiding this comment

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

Remove the not translated string here ("Appliquer", "Loading..")

Copy link

Choose a reason for hiding this comment

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

same here

@B-Rass B-Rass merged commit 51f15b4 into upgrade-to-bootstrap-4 Aug 7, 2024
1 check failed
@B-Rass B-Rass deleted the refactorize-visualization-capacity-and-infrastructure branch August 7, 2024 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants