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

Forrt Community Mapping #150

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

richarddushime
Copy link
Contributor

Hi @forrtproject/team-website

I'm working on the Forrt Community Mapping using Shinnyapps.io. Could you please check out the draft here: Forrt Community Mapping (Test)? I'd love your feedback and suggestions before finalizing it.

Thanks

@richarddushime richarddushime requested a review from a team as a code owner July 18, 2024 17:48
Copy link

This pull request has been inactive for more than 90 days. If there is no further activity, it will be automatically closed in seven days time. You can reopen the pull request if it is still relevant.

@github-actions github-actions bot added the stale Marked by labeler label Sep 17, 2024
@richarddushime
Copy link
Contributor Author

Will review it again soon

@richarddushime richarddushime removed the stale Marked by labeler label Sep 17, 2024
@flavioazevedo
Copy link
Contributor

Hi there @richarddushime ,
I love this <3

It is a really good start mapping the community's distribution across the globe!

The branch seems to have a conflict somewhere. Can you update it?

@flavioazevedo
Copy link
Contributor

ps: we should probably re-think "Forrt Mapping". We have a "Mapping OS" in our educational nexus, and we have "Community" in About menu. So perhaps we can call it "FORRT Network"? I am super open to ideas. I don't think my suggestion is great. So if anyone has better names @forrtproject/team-website , please feel free to chime in!

It would also be really nice if it opened in a new tab, rather than the same tab when you click on it (so that folks still have a tab open w/ our website).

@LukasWallrich
Copy link
Contributor

To open the app in a new window and mask the URL so that it looks like it is on forrt.org, we can follow the model I just added to the replication apps in #181.

Re name, I would suggest we call this community and replace the current community page - I am not sure if the current community page is helpful, as it is so incomplete and does not contain any information on the individuals displayed - and would require a fair bit of work to be more useful than this? If we want to keep the community, then maybe this can be called community map? I am not so keen on network, as that sounds more like partnerships to me?

@flavioazevedo
Copy link
Contributor

I like Community, and Community Map.

re: the community page: I totally see your point!

The reason we have it is that people needed to have a page where they can say I am a part of this. It preceeds the credit page, and it is useful for some members (but not the whole community). We can absolutely re-think the community page (and improve it) as part of team website!

@flavioazevedo
Copy link
Contributor

Dear @richarddushime , the branch says there are conflicts which maybe you know how to resolve?

@richarddushime
Copy link
Contributor Author

Dear @richarddushime , the branch says there are conflicts which maybe you know how to resolve?

Sure , I will resolve this ASAP
Thanks

@LukasWallrich
Copy link
Contributor

@richarddushime just pinging you again to resolve this conflict :) would be great to add to the page

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

3 participants