Thank you, Ahmet! Happy to help! Both changes [1] and [2] have been reviewed and merged by Alexey Romanenko.
We wanted to thank Alexey Romanenko, David Huntsperger, Pablo Estrada, Alya Boiko for reviewing and helping to contribute 52 enhancements, fixes and case study related additions for the Beam website in the last 6 months since July’22! [3] [1] https://github.com/apache/beam/pull/22221 [2] https://github.com/apache/beam/pull/24747 [3] https://github.com/apache/beam/pulls?page=1&q=is%3Apr+author%3Abullet03+is%3Aclosed+merged%3A%3E%3D2022-07-01 From: Ahmet Altay <al...@google.com> Date: Tuesday, January 3, 2023 at 2:22 PM To: Alex Kosolapov <alex.kosola...@akvelon.com> Cc: "dev@beam.apache.org" <dev@beam.apache.org>, Rebecca Szper <rsz...@google.com>, Bulat Safiullin <bulat.safiul...@akvelon.com>, Alexey Romanenko <aromanenko....@gmail.com>, Rajkumar Gupta <rajkumargu...@google.com> Subject: [EXTERNAL] Re: Beam Website Feedback Thank you Alex and Bulat for improving this. We all very much appreciate it. On Thu, Dec 22, 2022 at 9:21 AM Alex Kosolapov <alex.kosola...@akvelon.com<mailto:alex.kosola...@akvelon.com>> wrote: Hi all, We were preparing some improvements for check-links.sh script that is used for testing Apache Beam website links during the website build with Bulat (@bullet03<https://github.com/bullet03>). We saw several categories of link checks and error statuses: * 404 - actual incorrect links - fixed in [1] and [2] * Valid links that appear to the script as incorrect, e..g., 9xx status code for LinkedIn requiring authentication in LinkedIn, some GitHub documentation links, example links, some Meetup links, etc. We propose to add a “verified_list” to check_links.sh so that manually verified links can be skipped in testing. Current verified list includes 15 links based on review of most recent test review. Inconvenience of this approach is that a verified link may become outdated, and would require an update of the “verified_list” in check_links.sh. This approach implemented in [3]. [3] also contains check-links.sh improvements: * Added a function that checks and reports Apache Beam staging website links to prevent the production website from having links to staging * Added script checks and reports Apache Beam website absolute links (links of the form https://beam.apache.org/path) - relative links in the sources are preferred to properly build and review website staging * Added sorting any invalid links by their error code - this may be more convenient for reviewing output [4] - optionally, update absolute links to relative links so that a staging website more closely resembles the production website We submitted [3] and [4] for PR review and tagged Alexey Romanenko to kindly help with reviewing these PRs. Please share your comments about proposed approach in the PRs or list. [1] https://github.com/apache/beam/pull/24635 [2] https://github.com/apache/beam/pull/24744 [3] https://github.com/apache/beam/pull/22221 [4] https://github.com/apache/beam/pull/24747 Thank you, Alex From: Rebecca Szper via dev <dev@beam.apache.org<mailto:dev@beam.apache.org>> Reply-To: "dev@beam.apache.org<mailto:dev@beam.apache.org>" <dev@beam.apache.org<mailto:dev@beam.apache.org>>, Rebecca Szper <rsz...@google.com<mailto:rsz...@google.com>> Date: Wednesday, December 21, 2022 at 10:15 AM To: Ahmet Altay <al...@google.com<mailto:al...@google.com>> Cc: Alexey Romanenko <aromanenko....@gmail.com<mailto:aromanenko....@gmail.com>>, dev <dev@beam.apache.org<mailto:dev@beam.apache.org>>, Rajkumar Gupta <rajkumargu...@google.com<mailto:rajkumargu...@google.com>> Subject: [EXTERNAL] Re: Beam Website Feedback Our team doesn't maintain the Beam website infrastructure, but last time something like this came up, David said that there are consultants that work on this type of thing. He pinged @bullet03<https://github.com/bullet03> on the Beam ticket, who was able to help. On Tue, Dec 20, 2022 at 5:06 PM Ahmet Altay <al...@google.com<mailto:al...@google.com>> wrote: On Tue, Dec 20, 2022 at 1:12 PM Ahmet Altay <al...@google.com<mailto:al...@google.com>> wrote: On Tue, Dec 20, 2022 at 9:14 AM Alexey Romanenko <aromanenko....@gmail.com<mailto:aromanenko....@gmail.com>> wrote: Thanks Ahmet! I’d prefer to fix the links as you did and add the redirect from old one - perhaps, there are other similar links that have been changed in the same way. Thank you for the review. I fixed it, and added a redirect too. Btw, I’m not sure that we still check the broken links as it was before, iirc, but probably it would be a good idea to add such check before publishing a website. I agree. I also do not know about the state of this. It would be good to add that links checker again. Adding @Rebecca Szper<mailto:rsz...@google.com> - in case this is something she can fix or would know who could fix it. — Alexey On 20 Dec 2022, at 18:04, Ahmet Altay via dev <dev@beam.apache.org<mailto:dev@beam.apache.org>> wrote: I did a search and found a few places with the broken link. Correct links should be: https://beam.apache.org/get-started/resources/videos-and-podcasts/ I created a PR to update the website (https://github.com/apache/beam/pull/24733). I do not know if that is the best solution. As an alternative we could consider setting up a redirect for the old link. We do not know who else would be still linking to the old one. Ahmet On Tue, Dec 20, 2022 at 8:52 AM Alexey Romanenko <aromanenko....@gmail.com<mailto:aromanenko....@gmail.com>> wrote: Hi Rajkumar, Could you specify where (which page) this link was found? Thanks! — Alexey On 20 Dec 2022, at 10:08, Rajkumar Gupta via dev <dev@beam.apache.org<mailto:dev@beam.apache.org>> wrote: Hi Team, Just a minor point, while browsing the site I noticed that the link below is not working. Can you please check? https://beam.apache.org/documentation/resources/videos-and-podcasts Regards, Raj -- Rajkumar Gupta | Technical Solutions Engineer - Google Cloud Platform | rajkumargu...@google.com<mailto:rfol...@google.com> | +91-9223541460<tel:+91%2092235%2041460>
