-
Notifications
You must be signed in to change notification settings - Fork 9
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
Refactoring of New Project view in manager dashboard #665
Comments
@ofr1tz , We looked at the PR. We also looked at the issue. But we do not completely understand the use-case. From the issue, what we understood is;
We have a couple of questions;
|
Thanks a lot for looking into this! We are indeed working on a web client for MapSwipe with more/different types of projects. Currently, we have implemented:
Regarding your questions: We are planning to use a separate database for the web client and run a separate instance of the manager dashboard for the MapSwipe web app. This is the reason why we want to make the manager dashboard configurable in regard to the allowed project types. Modifications on the mobile app would thus only be necessary if a decision is taken to implement one of the new (web app) project types in the mobile app. |
@udaynwa thanks a lot for looking at the PR. Did @ofr1tz answers help to clarify things? We are also happy to set up a call together with @Hagellach37 (and anyone else interested) and give some more background information, if this is helpful? |
@masain , yes, it looks good to us. We're happy to have a call if need be. |
@danbjoseph @Hagellach37 @laurentS Should we merge this one? |
As presented in one of the last MapSwipe calls (recoding of the demo: MapSwipe web client demo by HeiGIT from 2023-02-14 community call ), we are currently developing a MapSwipe web client. The web client does not only make it possible to contribute from the browser and thus facilitate access to users who do not use an Android or iOS device or who are not allowed to install apps. It also opens up more possibilities to experiment with new project types. Moreover, we intent to implement more generic versions of the "classic" MapSwipe project type that may help widen the thematic scope of MapSwipe and may be used in crowdmapping projects beyond MapSwipe, in the context of entirely new thematic fields.
At the moment, we use a fork of the old manager dashboard for project creation and management that allows us to create projects of types that are not yet implemented in MapSwipe, and to add project properties that are needed for the more generic versions of the existing MapSwipe project types (e.g. we allow the project creator to define custom answers to BUILD_AREA projects).
However, rather than continuing with our own fork, we aim at integrating our changes to the MapSwipe manager dashboard and thus contribute to its development as well as benefit from the contributions of others.
In order to make it easier to add new project types, we suggest a refactoring of the New Project view of the manager dashboard into separate components for each project type. We would implement this in the following steps:
Create a BasicProjectInfoForm component for input fields that are shared among projects of all types and that will be reused in all type specific project creation components, e.g. project topic, project description
Create a tileServerForm for input fields regarding tile server (A and possibly B), zoom level etc. that are used for project creation components of projects that are based on image tiles
Create separate components for project creation for the different project types
Based on this refactoring as a prerequisite, we would contribute project creation components for our new experimental project types (i.e. either entirely new project types or adapted, more generic versions of existing project types). We also intent to make the manager dashboard configurable in regard to the allowed project types, so that the regular manager dashboard instance would only allow for creation of the project types currently supported by the MapSwipe mobile app, while another manager dashboard instance would enable creation of other project types (and write to a different Firebase instance).
The text was updated successfully, but these errors were encountered: