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.

Sign up for GitHub

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

Jump to bottom

Resources management refactoring #219

Draft
wants to merge 7 commits into
base: develop
Choose a base branch
from
Draft

Resources management refactoring #219

wants to merge 7 commits into from

Conversation

jantiegges
Copy link
Collaborator

@jantiegges jantiegges commented Feb 6, 2023 •
edited by n3rdc4ptn
Loading

Co-authored-by: Moritz Reich m.reich.1@campus.tu-berlin.de

jantiegges and others added 7 commits February 6, 2023 10:53
@jantiegges @n3rdc4ptn
minor ui changes
1c49dae 
Co-authored-by: Moritz Reich <n3rdc4ptn@users.noreply.github.com>
@jantiegges @n3rdc4ptn
minor fixes
dc513af 
Co-authored-by: Moritz Reich <m.reich.1@campus.tu-berlin.de>
@jantiegges @n3rdc4ptn
intermediate resource management refactoring
76c8f21 
Co-authored-by: Moritz Reich <m.reich.1@campus.tu-berlin.de>
@jantiegges @n3rdc4ptn
intermediate resource management refactoring
a22522e 
Co-authored-by: Moritz Reich <m.reich.1@campus.tu-berlin.de>
@n3rdc4ptn
start optimizing dropdownfilter
a743747 
Signed-off-by: Moritz Reich <m.reich.1@campus.tu-berlin.de>
@n3rdc4ptn
added identifiable
bdeb998 
Signed-off-by: Moritz Reich <m.reich.1@campus.tu-berlin.de>
@n3rdc4ptn
added resourcetable
2715244 
Signed-off-by: Moritz Reich <m.reich.1@campus.tu-berlin.de>
@n3rdc4ptn
Copy link
Contributor

There is too much change needed in the complete Resource Management.

This PR contains an approach how to optimize and improve the code quality there.
The main idea is:

  • separate the resource management in separate routes
  • Reduce Complexity in Resource Tables --> no need for so much components.
  • optimize code quality and remove usage of vanilla javascript daisyui modals --> Change to headlessui Dialog

@n3rdc4ptn n3rdc4ptn added wontfix This will not be worked on refactoring labels Feb 7, 2023
@jantiegges jantiegges changed the title UI Changes and resources management refactoring Resources management refactoring Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
refactoring wontfix This will not be worked on
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jantiegges @n3rdc4ptn