-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Final merge of website revamp #13871
Conversation
…ponent (#13331) * Implemented About social media component * Review: refactored social icons component & added licenses to svg icons
…d Changed Works with components (#13406) * Implemented About social media component * Implemented About social media component * Implemented Stay up to date component * Updated supported integrations component * Review: Added comments
… Header and Navbar (#13439) * Implemented About social media component * Implemented Quotes component * Implemented Hero component * Changed Navbar component * Added license * Review changes * Review changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sending this review to call attention to existing TODOs in website-revamp
Codecov Report
@@ Coverage Diff @@
## master #13871 +/- ##
==========================================
- Coverage 82.98% 82.84% -0.14%
==========================================
Files 469 466 -3
Lines 58298 57596 -702
==========================================
- Hits 48379 47718 -661
+ Misses 9919 9878 -41
Continue to review full report at Codecov.
|
Run Python PreCommit |
Run PythonLint PreCommit |
…ntact us pages, navbar on mobiles (#13511) * init * minor fixes * fix mobile navbar * delete br's * delete br * change header size * Update website/www/site/data/en/community_list.yaml Co-authored-by: Brian Hulette <hulettbh@gmail.com> * add content, change shortcode * Update website/www/site/content/en/community/contact-us.md Co-authored-by: Brian Hulette <hulettbh@gmail.com> * fixes/changes without content update * merge * whitespaces * Update website/www/site/static/js/section-nav.js Co-authored-by: Brian Hulette <hulettbh@gmail.com> * fixes * add attribute for link * whitespace * content update * examples update * fix: * add contact us copy * Update website/www/site/content/en/community/contact-us.md Co-authored-by: Brian Hulette <hulettbh@gmail.com> * update copy Co-authored-by: Brian Hulette <hulettbh@gmail.com>
…beam page and navigations (#13718) * init * fix * delete node modules * hero fix on mobiles * delete code_samples * changes' * Add copy to join beam * delete whitespace * fix * fix list * join beam * update copy of join bema * fix * Update website/www/site/content/en/community/join-beam.md Co-authored-by: Brian Hulette <hulettbh@gmail.com> * Update website/www/site/data/en/join_beam.yaml Co-authored-by: Brian Hulette <hulettbh@gmail.com> * Update website/www/site/data/en/join_beam.yaml Co-authored-by: Brian Hulette <hulettbh@gmail.com> * Update website/www/site/data/en/join_beam.yaml Co-authored-by: Brian Hulette <hulettbh@gmail.com> * Update website/www/site/data/en/join_beam.yaml Co-authored-by: Brian Hulette <hulettbh@gmail.com> * Update website/www/site/data/en/join_beam.yaml Co-authored-by: Brian Hulette <hulettbh@gmail.com> Co-authored-by: Brian Hulette <hulettbh@gmail.com>
…er, button bar, calendar, blog, blog post and documentation page (#13596) * all changes * added calendar * fix list order * fix calendar * external links * minor fix * changes without content * add license * Update website/www/site/static/js/calendar.js Co-authored-by: Brian Hulette <hulettbh@gmail.com> * last changes * filtering fix * content update * fix todos * fix todos * Update website/www/site/layouts/index.html Co-authored-by: Brian Hulette <hulettbh@gmail.com> * fixes * resolved conflict * icons * add copy * typo: * fixed blog * fixed icons on homepage * fix * icons fix Co-authored-by: Brian Hulette <hulettbh@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Jakub-Sadowski I noticed some strange behavior on the homepage when I switch to mobile view.
…de, become a committer pages and twitter feed (#13565) * contribution-guide * become a committer and twitter * fix * update files * add license * whitespaces * committer not commiter * uppercase * arrow-list template * fix double right navigation * delete br's * delete duplicated class * changes * fix typo * update content * newer copy * delete content * statics * content * content updated * fix typography: * fix of content * fix * content fix * highlight change * fix contribution * fix navbar * content update * fixes * change div * Update website/www/site/data/en/committer_traits.yaml Co-authored-by: Brian Hulette <hulettbh@gmail.com> * Update website/www/site/layouts/shortcodes/highlight.html Co-authored-by: Brian Hulette <hulettbh@gmail.com> Co-authored-by: Brian Hulette <hulettbh@gmail.com>
… matrix, powered by, beam practises, feedback component (#13858) * init * missing content * search styles * add link to feedback * content update * examples update * fix powered by * add page * fix * fix logos * add capability * change of sdks * Update website/www/site/layouts/partials/feedback.html Co-authored-by: Brian Hulette <hulettbh@gmail.com> * Update website/www/site/data/en/powered-by.yaml Co-authored-by: Brian Hulette <hulettbh@gmail.com> * fix content * fix matrix * Update website/www/site/data/en/powered-by.yaml Co-authored-by: Brian Hulette <hulettbh@gmail.com> * basics * fix * fix hero component: * fix mobile * fix navigation * navbar fix Co-authored-by: Brian Hulette <hulettbh@gmail.com>
aggregating elements per key and window | ||
* [_Flatten_](#implementing-the-flatten-primitive) - union of PCollections | ||
* [_Window_](#implementing-the-window-primitive) - set the windowing strategy | ||
for a PCollection |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note these links don't work. They refer to anchors in the runner contributioin guide where this content came from.
I don't know if I'm missing something, but the capability matrix link seems to be broken: http://apache-beam-website-pull-requests.storage.googleapis.com/13871/documentation/runners/capability-matrix/what-is-being-computed |
Yeah this is an issue with the staged website our tooling generates. It works correctly at http://test-beam.surge.sh/documentation/runners/capability-matrix/what-is-being-computed/ |
This PR is for the final merge of the website revamp into the master branch, which will publish it to https://beam.apache.org. This is the result of contributions generated by @bntnam and @Jakub-Sadowski, that were reviewed as PRs into the website-revamp branch.
Note there are some minor issues (broken links) with the staged version of this website generated by our infrastructure. There is a staged version that better reflects the final result at http://test-beam.surge.sh/
Post-Commit Tests Status (on master branch)
Pre-Commit Tests Status (on master branch)
See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.
GitHub Actions Tests Status (on master branch)
See CI.md for more information about GitHub Actions CI.