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

Clean-up the landing Readme.md #31

Open
2 tasks
EricBoix opened this issue Jul 26, 2021 · 4 comments
Open
2 tasks

Clean-up the landing Readme.md #31

EricBoix opened this issue Jul 26, 2021 · 4 comments
Assignees

Comments

@EricBoix
Copy link
Contributor

EricBoix commented Jul 26, 2021

  • some sections have FIXMEs
  • the section describing the modules could be the default menu
    that the application loads (and pointed in the docs)
@valentinMachado
Copy link
Contributor

valentinMachado commented Sep 13, 2021

resolve by this PR => VCityTeam/UD-Viz#223 ?

@EricBoix EricBoix assigned EricBoix and unassigned valentinMachado Sep 15, 2021
@EricBoix
Copy link
Contributor Author

@valentinMachado My understanding of UD-Viz is that it is a component (package) used by UD-Viz-Template whereas UD-Viz-Template is an application (using/build-on UD-Viz). Hence UD-Viz-Template/Readme.md should decribe how to derive your application which is information that doesn't belong into the UD-Viz package documentation. If your comment doesn't fall into this viewpoint, could you please be more specific about what in this VCityTeam/UD-Vuiz#223 PR solves the present issue ? (because that PR is a quite heavy and it modifies 50+ files)

@valentinMachado
Copy link
Contributor

@EricBoix To be honest I don't remember why I thought this PR was solving this...

You're right the point of this repository is to propose to an ud-viz user a template architecture for its application.
Since ud-viz has quite changed with the version 3.0.0 (https://github.com/VCityTeam/UD-Viz/releases/tag/v3.0.0) and now includes 3 packages (shared/browser/node), this repository needs to be refacto as well.
This issue should be resolved after/while this one #44

I am currently working on this issue (VCityTeam/UD-Imuv#406), to adapt the application UD-IMUV with the new ud-viz version, which is more or less the same work. We should then be able to refacto this repository, so the user can implements an ud multiplayer game (as UD-IMUV) or an usual application (as AllWidget template). We could also think to include test pattern, devel workflow, documentation generation, coding style like on ud-viz.

@EricBoix
Copy link
Contributor Author

EricBoix commented May 5, 2023

My understanding is that @mathieuLivebardon is currently working on this issue as part of this branch. @mathieuLivebardon Since I contributed to the original version don't hesitate to solicit my input on this matter when reviewing the upcoming PR.

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

No branches or pull requests

2 participants