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

feat: reduce navi and add community information #215

Merged
merged 4 commits into from
Nov 16, 2021

Conversation

danielpeintner
Copy link
Contributor

@danielpeintner
Copy link
Contributor Author

See screenshot below. I think it is best to show you the result during the next call.

grafik

@egekorkan
Copy link
Contributor

Do we now have place for documentation and/or videos ?

@danielpeintner
Copy link
Contributor Author

Do we now have place for documentation and/or videos ?

Yes, but maybe it is best to to show it during the next call?

grafik

@egekorkan
Copy link
Contributor

In the call of 02.11, there were the following comments:

  • Standards sounds like standardization documents
  • Resolution was to use the word Groups and also put the Community Group in there.

@danielpeintner can you incorporate these changes?

@danielpeintner
Copy link
Contributor Author

danielpeintner commented Nov 8, 2021

@danielpeintner can you incorporate these changes?

I updated the PR accordingly

grafik

Note: Community now also contains some information like Twitter/SO .. not sure what we want to do with it.

Comment on lines 7 to 13
<h2>How to get in touch</h2>
<ul>
<li><a href="https://twitter.com/W3C_WoT">Twitter</a></li>
<li><a href="https://stackoverflow.com/questions/tagged/web-of-things">Stack Overflow</a></li>
<li>GitHub Discussions (<mark>TODO</mark>)</li>
</ul>

Copy link
Contributor Author

@danielpeintner danielpeintner Nov 9, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: I don't think this PR is ready to be merged until we solve the section above

@@ -1,9 +1,16 @@
---
layout: default
title: W3C WoT – Community Groups
title: W3C WoT – Community
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shall we keep "Groups" in the title?

docs/_config.yml Outdated
url: wg/
- title: Interest Group
url: ig/
- title: Community
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shall we keep "Groups" in the navigation?

@danielpeintner danielpeintner marked this pull request as draft November 9, 2021 08:43
@egekorkan
Copy link
Contributor

My feedback:

  • "How to get in touch" should be in another tab like "Contact" to the right of Videos
  • "Community" should be named "Community Groups" to be aligned with the other items
  • Contact information should include Public Emailing lists, IRC, and mentioning that GitHub issues can be used for contacting

@egekorkan
Copy link
Contributor

From @ashimura : we should say that the CGs are related. They are not part of the WoT work officially.
From @relu91 : we should include the Web Thing Protocol Community Group

@danielpeintner
Copy link
Contributor Author

  • "How to get in touch" should be in another tab like "Contact" to the right of Videos

  • "Community" should be named "Community Groups" to be aligned with the other items

Done

  • Contact information should include Public Emailing lists, IRC, and mentioning that GitHub issues can be used for contacting

The contact page might need more work. Proposals what should be in there?

grafik

@danielpeintner danielpeintner marked this pull request as ready for review November 9, 2021 13:16
@egekorkan egekorkan merged commit 6527822 into w3c:main Nov 16, 2021
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