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 clean up #107

Open
30 tasks done
EmilieSonne opened this issue Dec 2, 2021 · 1 comment
Open
30 tasks done

Code clean up #107

EmilieSonne opened this issue Dec 2, 2021 · 1 comment

Comments

@EmilieSonne
Copy link
Collaborator

EmilieSonne commented Dec 2, 2021

  • fjern unused code/UI
  • kommentarer
  • giv mulighed for at lave " og linjeskift, når man inputter tekst
  • fejlhåndtering
    • CheckCategoryOrder på visitor application
    • check om det vi loader fra databasen er "valid" indhold inden det smides i objekterne (evt. validation i setterne?)
    • check om det er legal inden vi lægger indhold på det databasen (har vi måske? - lav en oversigt/graf)
    • ret fejlbesked i _Host til dansk besked om at der ikke er forbindelse til serveren.
  • kør resharper clean up igen... (både management og visitor)
  • fjern alt med authentication + den anden solution i main
  • Skriv noget med at " // swallow attempt to save invalid data " i vores tomme if () i edit-modaler

  • ryd op
  • omdøb mappe fra functions -> function
  • fjern informationboard som input param i methods der kaldes på et information board (det er 'this')
  • fjern usings
  • ret rækkefølge på props and stuff
  • access modifiers
  • få rettet det sådan at GetInformationBoards ligger på InformationBoardViewModel etc., så vi har en clean DbService class (bare overfør _dbService via InitializeLicense)
  • læg viewmodelsne i model-mappen (de hører til model component)
  • flyt list< IconOptions >
  • styr på hvad der egentlig skal være i constructorerne (fx i InformationBoardViewModel laver vi en ny model, men det mener jeg ikke, vi gør i de andre - why?)
  • større ikoner på rediger og slet posts
  • Flyt delete-methods et niveau op; delete category skal ligge på information board: 1. Delete from database, 2. Delete from Categories-dictionary, 3. Delete from Category Order (BETYDER AT VI KAN FJERNE NOGLE CALL BACK JOOO, vi skal bare stadig re-fetche ved add categories. Kan vi evt. bare gøre det i AddCategory metoden i stedet for I call back'en så vi kun har statehaschanged?)
    • post og category delete
    • når alle vores viewmodels bliver smidt ind i model er vi nødt til at skelne mellem 'data' og 'model', for 'model' kommer til at indeholde mere, så 'model' = 'data model' og 'view model' = model... fuck os
  • pil på nederste kategori i edit categories skal rykkes op og stå i toppen af dens linje
  • byt om på rækkefølge på add category og edit categories buttons
  • implementer at når man trykker annuller i rediger-menuer, så annulleres der rent faktisk.. lav et mellem-objekt eller noget? :i måske virker det faktisk, men få lige styr på det
  • sæt "regler" for post indhold på edit-delen også (lige nu kan man slette titlen, hvis man går in i edit) (mangler edit category)
  • hvis _selectedCategory == null => SelectedCategory = Categories[CategoryOrder.first()]; på InformationBoardModel
@EmilieSonne EmilieSonne changed the title fjern unused code/UI Code clean up Dec 2, 2021
@EmilieSonne
Copy link
Collaborator Author

EmilieSonne commented Dec 7, 2021

Nye ting:

  • tilføj predicates til alle update methods
  • når en category slettes, skal tilhørende posts lægges i bin og slettes efter 31 dage (evt. kunne vi starte med en empty-bin-button) (skal laves som event på category, som BinService subscriber til, som invokes i delete category)
  • flyt conversion methods over I conversion service I functions
  • validation på link på post
  • fix den der editform på redigersiderne - har det noget med bind-value:event="oninput" at gøre? (category, informationboard, post - vi vil have de jävle warnings frem!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants