Add seeders for development #206
Conversation
Co-authored-by: Luís Duarte <[email protected]>
|
Check the documentation preview: https://666b2ff156f07c79c91b0390--niaefeup-backend-docs.netlify.app |
|
Check the documentation preview: https://6674664aff107b90fe3e52c7--niaefeup-backend-docs.netlify.app |
|
Check the documentation preview: https://67128f5518a026754a930d08--niaefeup-backend-docs.netlify.app |
|
Check the documentation preview: https://6738d8b94d832f968e8c51f1--niaefeup-backend-docs.netlify.app |
|
Check the documentation preview: https://6738d9c423d885a842a0bd1b--niaefeup-backend-docs.netlify.app |
|
Check the documentation preview: https://6745c4db330f675ec1cb9751--niaefeup-backend-docs.netlify.app |
| fun checkSeedArgument() { | ||
| try { | ||
| val seedProperty = System.getProperty("seed") | ||
| if (seedProperty == "true") { | ||
| seed = true | ||
| } | ||
| } catch (_: NullPointerException) { } catch (_: IllegalArgumentException) {} | ||
| } |
There was a problem hiding this comment.
This function is not being used we can remove it
|
|
||
| override fun run(args: ApplicationArguments?) { | ||
| logger.info("Running Startup hook...") | ||
| // checkSeedArgument() |
There was a problem hiding this comment.
Can remove here as well
| val debug: Boolean = true // false | ||
|
|
||
| var seed: Boolean = true // false |
There was a problem hiding this comment.
I think both of these should be false here by default
| @Value("\${app.debug}") | ||
| val debug: Boolean = true // false | ||
|
|
||
| var seed: Boolean = true // false |
There was a problem hiding this comment.
| @Value("\${app.debug}") | |
| val debug: Boolean = true // false | |
| var seed: Boolean = true // false | |
| @Value("\${app.debug:false}") | |
| val debug: Boolean = false | |
| @Value("\${app.seed:false}") | |
| private var configSeed: Boolean = false | |
| val seed: Boolean | |
| get() = System.getProperty("seed")?.toBooleanStrictOrNull() ?: configSeed |
There was a problem hiding this comment.
This would allow us to either specify app.seedon application.properties, or override whatever its value is (even null) if we run bootRun with ./gradlew bootRun -Pseed
There was a problem hiding this comment.
It also defaults to if app.debugor app.seed are missing they are assumed false
| @@ -1,3 +1,6 @@ | |||
| # Application Settings | |||
| app.debug = true | |||
|
|
|||
There was a problem hiding this comment.
| app.seed = true |
| // Role: Admin/Settings Access Only | ||
| // This role grants users administrative access to settings and superuser functionalities, but not permissions to manage accounts or activities. | ||
| PerActivityRole( | ||
| permissions = Permissions(setOf(Permission.EDIT_SETTINGS, Permission.SUPERUSER)) | ||
| ) |
There was a problem hiding this comment.
This is a repeat from above
| perActivityRoles.map { entityManager.merge(it) }.toMutableList() | ||
|
|
||
| val projects = (1..6).map { | ||
| val title = faker.company().bs() |
| .replace(" ", "-") | ||
| .replace(Regex("[^a-z0-9-]"), "") | ||
|
|
||
| // Ensure only one TimeLineEvent per project |
| val roles = listOf( | ||
| Role( | ||
| name = "President", | ||
| isSection = true, | ||
| permissions = Permissions(listOf(Permission.EDIT_ACTIVITY)) | ||
| ), | ||
| Role( | ||
| name = "Vice President", | ||
| isSection = true, | ||
| permissions = Permissions(listOf(Permission.EDIT_ACTIVITY)) | ||
| ), | ||
| Role( | ||
| name = "Member", | ||
| isSection = false, | ||
| permissions = Permissions(listOf(Permission.VIEW_ACTIVITY)) | ||
| ) | ||
| ) |
There was a problem hiding this comment.
I think there should be a couple more sections (isSection = true) to test the frontend, and maybe make them match the frontend for it to be less confusing! (Aka Direção Membros Recrutas Alumni)
| @@ -6,6 +9,7 @@ spring.datasource.password= | |||
| # Spring JPA | |||
| spring.jpa.hibernate.ddl-auto=update | |||
| spring.jpa.open-in-view=false | |||
| spring.jpa.database-platform=org.hibernate.dialect.H2Dialect | |||
|
Btw I can't approve/disapprove because I opened the PR ;( |
Review checklist