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

update showcase tree structure #774

Merged
merged 24 commits into from
Oct 31, 2023
Merged

Conversation

agitana
Copy link
Member

@agitana agitana commented Oct 31, 2023

No description provided.

@agitana agitana requested a review from a team as a code owner October 31, 2023 00:26
@MauricioUyaguari
Copy link
Member

I don't think we need subfolders under Build a data model.
I think for this we should have one or 2 showcases with a detailed usage of Class, Associations, Enum and a Diagram.
If we need to do a further split down we could in the future but I think for now thats fine.

@MauricioUyaguari
Copy link
Member

Lets remove the serializable folder please.

@MauricioUyaguari
Copy link
Member

text elements don't really interact with the model. we can just rename that folder to
Use text element

@MauricioUyaguari
Copy link
Member

For now external format is fine, we should do another walkthrough. As
Basically we should probably have it something like

  • External Format
    • Types
    • Writing Internalize Func (Reading External Format)
    • Writing Externalize Func (Writing to External Format)

@agitana
Copy link
Member Author

agitana commented Oct 31, 2023

thanks @MauricioUyaguari actioned your feedback can you please review?

@MauricioUyaguari MauricioUyaguari merged commit 8083c3d into master Oct 31, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants