Skip to content

Change the Code Examples to match the Readme examples #49

Closed

Description

@Mom0aut

The current code examples do not match the Readme Code-Examples and it would be simpler to follow if they are using the same patterns.

Current Examples:
https://github.com/Mom0aut/DynamicRelations/blob/master/testing/src/main/java/at/test/drm/AnnotationTest.java

  • AnnotationTest1-3

This should be refactored to be named like at the Readme File:

  • PersonEntity
  • DogEntity
  • etc..

Activity

  1. changed the title [-]Change the Code-Examples to match the Readme examples[/-] [+]Change the Code Examples to match the Readme examples[/+] on Aug 24, 2025
  2. cs168898 commented on Aug 28, 2025

    @cs168898
    Contributor

    Hello, this is my first time contributing, is it okay if I take this issue? And to clarify, to resolve this issue is to change the class names, file names and import names? Thank you!

  3. Mom0aut commented on Aug 28, 2025

    @Mom0aut
    OwnerAuthor

    Hey sure go ahead 🚀

    Yes the EntityClasses/Packages should be renamed to better fit the Readme-File

  4. cs168898 commented on Aug 29, 2025

    @cs168898
    Contributor

    Hi, can i check if the AnnotationTestRelationDao and AnnotationTestRelation is implemented in the 'testing' directory? Because from my fork it seems like it is not, and if it is not, should I implement it?

  5. Mom0aut commented on Aug 29, 2025

    @Mom0aut
    OwnerAuthor

    Hey you dont need to implement the Dao-Methods from Spring. The Spring-Framework does this for you 😄

    Here is an simple tutorial about Spring Daos: https://www.baeldung.com/java-dao-pattern

  6. cs168898 commented on Aug 29, 2025

    @cs168898
    Contributor

    I see.. Alright, I have created a pull request, let me know if it is what you wanted. If there are any mistakes or any changes, please let me know!

  7. Mom0aut commented on Aug 29, 2025

    @Mom0aut
    OwnerAuthor

    There are still some tests failing, you can verify your changes when you run mvn clean install
    The current error from the tests:

    Image
  8. Mom0aut commented on Sep 2, 2025

    @Mom0aut
    OwnerAuthor

    merged with #52 🚀

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

Metadata

Metadata

Assignees

Labels

documentationImprovements or additions to documentationenhancementNew feature or requestgood first issueGood for newcomershelp wantedExtra attention is needed

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions