open-feature / docs.openfeature.dev

OpenFeature Documentation
https://openfeature.dev
Creative Commons Attribution 4.0 International
6 stars 19 forks source link

feat: add Community repo as content #288

Closed aepfli closed 1 year ago

aepfli commented 1 year ago

We are adding the community repo as content. This way we do have a better overview, and might structure our community guidelines more efficient.

netlify[bot] commented 1 year ago

Deploy Preview for lucky-creponne-baf9b3 ready!

Name Link
Latest commit 833f3006aadfbf770029e66f832543e51f53781b
Latest deploy log https://app.netlify.com/sites/lucky-creponne-baf9b3/deploys/640b46db15c26b00087e27e2
Deploy Preview https://deploy-preview-288--lucky-creponne-baf9b3.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

aepfli commented 1 year ago

based on docusaurus our community repo has some issues - if this is something we want to continue with, i will fix them

- On source page path = /community/:
   -> linking to social-media (resolved as: /community/social-media)
   -> linking to ./LICENSE (resolved as: /community/LICENSE)

- On source page path = /community/branding-guidelines:
   -> linking to assets/logo/horizontal (resolved as: /community/assets/logo/horizontal)
   -> linking to assets/logo/horizontal/black (resolved as: /community/assets/logo/horizontal/black)
   -> linking to assets/logo/horizontal/white (resolved as: /community/assets/logo/horizontal/white)
   -> linking to assets/logo/stacked (resolved as: /community/assets/logo/stacked)
   -> linking to assets/logo/stacked/black (resolved as: /community/assets/logo/stacked/black)
   -> linking to assets/logo/stacked/white (resolved as: /community/assets/logo/stacked/white)
   -> linking to assets/logo/icon (resolved as: /community/assets/logo/icon)
   -> linking to assets/logo/icon/black (resolved as: /community/assets/logo/icon/black)
   -> linking to assets/logo/icon/white (resolved as: /community/assets/logo/icon/white)
beeme1mr commented 1 year ago

I think this is a great idea. Please feel free to adjust the Community markdown files to make the linter happy 😄

toddbaert commented 1 year ago

@aepfli I've approved https://github.com/open-feature/community/pull/159, hopefully that unblocks you here.

aepfli commented 1 year ago

another pr to improve the layouting - after this one, i think we can update the submodule and get this merged - https://github.com/open-feature/community/pull/160

aepfli commented 1 year ago

@beeme1mr @toddbaert this is now ready for review - be aware this was not about making the content beautiful, but making it visible.

image

another improvement i already see is removing the Readme.md from the asset folder :)

beeme1mr commented 1 year ago

Hey @aepfli, I've opened this PR to fix some typos and improve the overall structure of the community content in the docs.

https://github.com/open-feature/community/pull/162

Thanks for putting this together. I think this will be very helpful and end up being the best way to find community content.