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

Enhancement: Add Education Section to Student Application Endpoint Response #92

Commits on Aug 11, 2023

  1. Configuration menu
    Copy the full SHA
    254b53f View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    1906dde View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    92de07f View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    2b5fc17 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    52758a5 View commit details
    Browse the repository at this point in the history
  6. ♻️ Make class the default export.

    Why? It helps encourage the one model one file approach.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    a15ab03 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    f18a75a View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    bc90d5d View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    4341ef2 View commit details
    Browse the repository at this point in the history
  10. ♻️ Implemented controller instance per request usage.

    This gives us access tonne of values techniques, like shared param extractors and dependent object lookups.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    31a10fe View commit details
    Browse the repository at this point in the history
  11. Configuration menu
    Copy the full SHA
    8f46ce6 View commit details
    Browse the repository at this point in the history
  12. Configuration menu
    Copy the full SHA
    da3e01d View commit details
    Browse the repository at this point in the history
  13. Configuration menu
    Copy the full SHA
    c057e98 View commit details
    Browse the repository at this point in the history
  14. Configuration menu
    Copy the full SHA
    4313942 View commit details
    Browse the repository at this point in the history
  15. Configuration menu
    Copy the full SHA
    181c81a View commit details
    Browse the repository at this point in the history
  16. Configuration menu
    Copy the full SHA
    3a2fd87 View commit details
    Browse the repository at this point in the history
  17. Configuration menu
    Copy the full SHA
    8ec7216 View commit details
    Browse the repository at this point in the history
  18. Configuration menu
    Copy the full SHA
    04d4da6 View commit details
    Browse the repository at this point in the history
  19. Configuration menu
    Copy the full SHA
    146653a View commit details
    Browse the repository at this point in the history
  20. 🚧 Return institution info in serializer.

    Next I need to return the study area and program details.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    a1610bb View commit details
    Browse the repository at this point in the history
  21. Configuration menu
    Copy the full SHA
    ca16958 View commit details
    Browse the repository at this point in the history
  22. Configuration menu
    Copy the full SHA
    09dff96 View commit details
    Browse the repository at this point in the history
  23. 🎨 Match application interface with database null constraints.

    It seems that mostly boolean values are required.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    3ee614a View commit details
    Browse the repository at this point in the history
  24. Configuration menu
    Copy the full SHA
    ad0e3a0 View commit details
    Browse the repository at this point in the history
  25. ♻️ Drop program hash and just return details as name.

    Why? the program table appears to basically be a constant.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    32e5b36 View commit details
    Browse the repository at this point in the history
  26. Configuration menu
    Copy the full SHA
    3e15e5c View commit details
    Browse the repository at this point in the history
  27. Configuration menu
    Copy the full SHA
    ce2969d View commit details
    Browse the repository at this point in the history
  28. Configuration menu
    Copy the full SHA
    64d494e View commit details
    Browse the repository at this point in the history
  29. Configuration menu
    Copy the full SHA
    311b7e8 View commit details
    Browse the repository at this point in the history
  30. Configuration menu
    Copy the full SHA
    1ac965a View commit details
    Browse the repository at this point in the history
  31. Configuration menu
    Copy the full SHA
    d2639d2 View commit details
    Browse the repository at this point in the history
  32. Configuration menu
    Copy the full SHA
    9a010a3 View commit details
    Browse the repository at this point in the history
  33. Configuration menu
    Copy the full SHA
    655db9a View commit details
    Browse the repository at this point in the history
  34. Configuration menu
    Copy the full SHA
    9d7328a View commit details
    Browse the repository at this point in the history
  35. Configuration menu
    Copy the full SHA
    0fc9790 View commit details
    Browse the repository at this point in the history
  36. ♻️ Use shared constant for CSFA amounts.

    These values are defined on the front-end, and it might be a good idea to serve them up via an endpoint at some point. Or replace them with an entirely different concept.
    
    Such as performing all this logic on the front-end.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    33a5f1e View commit details
    Browse the repository at this point in the history
  37. Configuration menu
    Copy the full SHA
    557d60c View commit details
    Browse the repository at this point in the history
  38. Configuration menu
    Copy the full SHA
    a6bdaa2 View commit details
    Browse the repository at this point in the history
  39. Configuration menu
    Copy the full SHA
    13e2123 View commit details
    Browse the repository at this point in the history
  40. Configuration menu
    Copy the full SHA
    15d45e9 View commit details
    Browse the repository at this point in the history
  41. Configuration menu
    Copy the full SHA
    c23b5bb View commit details
    Browse the repository at this point in the history
  42. Configuration menu
    Copy the full SHA
    1cbe4db View commit details
    Browse the repository at this point in the history
  43. Configuration menu
    Copy the full SHA
    a3a3e46 View commit details
    Browse the repository at this point in the history
  44. 🔥 Drop now redundant manual reference to schema.

    Schema is now set via config; although, it can be overridden at will.
    
    I might end up breaking this out an applying it per-commit?
    Otherwise the commits that introduced this will be broken.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    699b524 View commit details
    Browse the repository at this point in the history
  45. Configuration menu
    Copy the full SHA
    e694b18 View commit details
    Browse the repository at this point in the history
  46. Configuration menu
    Copy the full SHA
    e8ad758 View commit details
    Browse the repository at this point in the history
  47. Configuration menu
    Copy the full SHA
    8397d72 View commit details
    Browse the repository at this point in the history
  48. Configuration menu
    Copy the full SHA
    1ff078c View commit details
    Browse the repository at this point in the history
  49. Configuration menu
    Copy the full SHA
    500348f View commit details
    Browse the repository at this point in the history
  50. Configuration menu
    Copy the full SHA
    2ca7980 View commit details
    Browse the repository at this point in the history
  51. Configuration menu
    Copy the full SHA
    782f645 View commit details
    Browse the repository at this point in the history
  52. Configuration menu
    Copy the full SHA
    68b1ede View commit details
    Browse the repository at this point in the history
  53. ♻️ Simply person details generation.

    How? Use a Person object instead of a Student.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    e6b9c55 View commit details
    Browse the repository at this point in the history
  54. Configuration menu
    Copy the full SHA
    83b7034 View commit details
    Browse the repository at this point in the history
  55. 🪲 Fix bug in student retrieval.

    I wasn't returning ids for sub-objects as part of the parent object.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    5a2c217 View commit details
    Browse the repository at this point in the history
  56. Configuration menu
    Copy the full SHA
    d65c74c View commit details
    Browse the repository at this point in the history
  57. Configuration menu
    Copy the full SHA
    a34641a View commit details
    Browse the repository at this point in the history
  58. Configuration menu
    Copy the full SHA
    f3c59ef View commit details
    Browse the repository at this point in the history
  59. Configuration menu
    Copy the full SHA
    6a7264b View commit details
    Browse the repository at this point in the history
  60. Configuration menu
    Copy the full SHA
    471ee49 View commit details
    Browse the repository at this point in the history
  61. Configuration menu
    Copy the full SHA
    444b86c View commit details
    Browse the repository at this point in the history
  62. Configuration menu
    Copy the full SHA
    63ec4dd View commit details
    Browse the repository at this point in the history
  63. Configuration menu
    Copy the full SHA
    b0069cc View commit details
    Browse the repository at this point in the history
  64. 🪲 Fix join condition between person and sex and language.

    A person might not have a sex or language so these need to be left joins to avoid excluding valid person data.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    ab85ffa View commit details
    Browse the repository at this point in the history
  65. ♻️ Rename addresses assocation on person to personAddresses.

    Why? To match the table name. Yeah its redundant, but I think simplicity is best for now.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    30583da View commit details
    Browse the repository at this point in the history
  66. Configuration menu
    Copy the full SHA
    29b45d8 View commit details
    Browse the repository at this point in the history
  67. Configuration menu
    Copy the full SHA
    43564b7 View commit details
    Browse the repository at this point in the history
  68. 🎩 Add residency to ERD.

    I haven't been adding all the relationships, just the ones I've used so far.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    7642975 View commit details
    Browse the repository at this point in the history
  69. Configuration menu
    Copy the full SHA
    9333799 View commit details
    Browse the repository at this point in the history
  70. 🚚 Rename funding request to funding sources.

    Why? To make room for actual funding requests section.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    d259888 View commit details
    Browse the repository at this point in the history
  71. Configuration menu
    Copy the full SHA
    31299f0 View commit details
    Browse the repository at this point in the history
  72. Configuration menu
    Copy the full SHA
    2b6c817 View commit details
    Browse the repository at this point in the history
  73. Configuration menu
    Copy the full SHA
    11e3182 View commit details
    Browse the repository at this point in the history
  74. 🔨 Add plantuml server to development environment.

    Why? So I can generate the massive ERD that this project uses.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    aa17ef6 View commit details
    Browse the repository at this point in the history
  75. Configuration menu
    Copy the full SHA
    fd7f023 View commit details
    Browse the repository at this point in the history
  76. Configuration menu
    Copy the full SHA
    424ffda View commit details
    Browse the repository at this point in the history
  77. 🔧 Change default schema for test setup.

    Why? To make use of the database schema restore in db/initializers/setup.sql.
    I was originally hoping to move everything to the default dbo schema, but this seems like it will be too big a problem to solve for now.
    
    Instead I'm opting to use a database restore, then truncate all tables for test.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    57023d8 View commit details
    Browse the repository at this point in the history
  78. Configuration menu
    Copy the full SHA
    3292155 View commit details
    Browse the repository at this point in the history
  79. Configuration menu
    Copy the full SHA
    28eb33a View commit details
    Browse the repository at this point in the history
  80. Configuration menu
    Copy the full SHA
    ad22ab2 View commit details
    Browse the repository at this point in the history
  81. Configuration menu
    Copy the full SHA
    165e1db View commit details
    Browse the repository at this point in the history
  82. Configuration menu
    Copy the full SHA
    2bdbe41 View commit details
    Browse the repository at this point in the history
  83. ✨ Return application draft application json as object instead of string.

    Why? Its much easier to work with an manipulate this way.
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    388bda0 View commit details
    Browse the repository at this point in the history
  84. ✨ Add education history to student application blob.

    This seems like something that will need to be backed by the education table at some point?
    klondikemarlen committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    3249f4f View commit details
    Browse the repository at this point in the history