algosup / 2023-2024-project-5-flutter-team-3

5th project of the year 2023-2024
MIT License
7 stars 0 forks source link

Document report: [Technical specification] #10

Closed Clementine951 closed 3 months ago

Clementine951 commented 4 months ago

Description

  1. Table of content
    1. Too low
    2. Twice a "5 in the Part III
      1. Pages and Authentication
    3. Authentication has only a subtitle
      1. A but no B
  2. Start of the document
    1. Document versions and information part are "useless"
    2. GitHub is already here for the control versioning
  3. Glossary
    1. Empty glossary
    2. To add
      1. API
      2. SDK
      3. ...
  4. Document purpose
    1. Repetitions and missing clarity (from my point of view, maybe not the case for others)
    2. The role of the technical is to decide how the functional will be implemented
    3. The scope and the objectives should be defined on the functional not the technical
    4. In the case these elements are not in the functional, it's a good thing
  5. Project definition
    1. You previously recommend to read the functional which is a good thing
    2. So, don't put functional parts on your technical
  6. Working environment
    1. The version of the SDK should be added
    2. Latest isn't specific enough if the document is read after a new release
  7. Minimum required version
    1. Good point for the minimum version
  8. III - 2 Database
    1. why is nothing on the lower part of the boxes?
  9. Table details and usage
    1. What is the difference between id and userId?
  10. C
    1. Are the matches going to be saved?
  11. Todo
    1. Data structurations
    2. Ways to interact
  12. 3 A Colors
    1. Should be on the functional (OK if it's not in it)
    2. Links to the graphic charter are not working (x2)
    3. In the sentence "good usage of " is the "" normal?
  13. 3 B Typo
    1. Same as the colours, should be in the functional with the design choices
    2. But again ok if not on the functional
  14. 4 widgets
    1. Which is the best option to implement them, if multiple possibilities
    2. In Text field: What is the maximum length and why?
    3. Why no minimum length?
    4. Maximum and minimum length for password also?
    5. In Tag Field: why" _" in Tag _Field?
  15. 4 B Icon button
    1. Backslash "\" at the end of the description
    2. Navbar is the generic word, think about using footer instead to avoid confusion with header
  16. 5
    1. A lot of things should be in the functional but as always, if not in the functional seems ok
    2. But pretty clear, know which type of element it is
    3. The footer is only visible on a few pages, is it normal or a thing you forgot?
  17. 5 subscription
    1. Miss the check for the data privacy, etc.
  18. 5 Settings
    1. According to the GDPR
      1. Get the data the app has about you
      2. Delete these data
    2. From where can you access the settings? (Only the swiping page?)
  19. 5 Job seeker profile
    1. The first time the footer is visible on the capabilities
  20. 5 authentication
    1. Only A and no B
  21. Part 6
    1. B to do
  22. Part 7
    1. Pretty clear maybe more information?
  23. Part 8 to do

Severity

  1. Medium (Should be fixed soon)

Version

version 0.02

Expected Correction

This review has been done knowing that the document isn't finished yet

Other Details

No response

Clementine951 commented 4 months ago

With the commit : f7c0be1

Clementine951 commented 3 months ago

This issue has been closed because the remaining issues have been moved in the issues #15 and #16. This has been done for more clarity for the assignees.