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

Refactoring #24

Open
fabiankuenzer opened this issue Jul 12, 2024 · 0 comments
Open

Refactoring #24

fabiankuenzer opened this issue Jul 12, 2024 · 0 comments

Comments

@fabiankuenzer
Copy link
Contributor

Vorschläge zur Verbesserung der Codequalität:

  • In der taskpane.ts wird unter Office.onReady() die gesamte Logik beschrieben. Teilweise ist diese schon in eigenen Funktionen ausgelagert. Im besten Fall werden hier perspektivisch nur noch Funktionen aufgerufen, die wiederum sinnvoll in eigene Files sortiert werden, sodass man einen guten Gesamtüberblick hat.
  • Die Branches feature/flaticon-integration und feature/logo haben jeweils ähnliche Funktionalitäten zum Einfügen von Base64 Images. Während oder nach dem Mergen sollten diese konsolidiert werden.
  • Linting wurde eingerichtet, allerdings wird es zurzeit nicht einheitlich genutzt, was unter Umständen zu Merge Konflikten führen kann. Das sollte entsprechend überall wie konfiguriert genutzt werden.
  • Es gibt einige Dependencies, die Vulnerabilities aufweisen. Die sollten entsprechend geupdatet werden.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant