Hannah, We don't have an exact date, but we are hoping to address all the comments and merge the PR by Wednesday. Will it be possible for you to wait until then?
On Thu, Apr 30, 2020 at 4:29 PM Hannah Jiang <hannahji...@google.com> wrote: > Since we want to move forward with the PR, I would like to ask the >> community to hold off changes to the current Beam website for a week, until >> we are able to review and merge the PR. Is this acceptable to everyone? > > Do we have an exact date when we can push changes to the website? I have > PRs to update documents so would like to plan ahead. > > On Thu, Apr 30, 2020 at 1:17 PM Nam Bui <nam....@polidea.com> wrote: > >> Hey guys, >> >> I tried my best to handle renamed files in Git. I have no clue why GitHub >> doesn't show it, but finally, I made this commit [1] (thanks for your >> idea @bhulette) so you guys can review changes with ease (there is no bunch >> of deleted markdown files anymore :D). Also, new staged version is >> deployed, you could check it out [2]. >> >> In case you are interested in translation, here is the proof of concept >> [3] (the earth icon on the right corner is temporarily used for switching >> languages). You can take a look at the translation guide for this PoC [4]. >> >> [1] >> https://github.com/apache/beam/pull/11554/commits/b267bb360866a723ac2536f408f23de648c7cd4d >> [2] >> http://apache-beam-website-pull-requests.storage.googleapis.com/11554/index.html >> [3] https://safe-relation.surge.sh/ >> [4] >> https://github.com/PolideaInternal/beam/blob/website-develop/website/CONTRIBUTE.md#translation-guide >> >> >> On Thu, Apr 30, 2020 at 7:24 PM Brian Hulette <bhule...@google.com> >> wrote: >> >>> Changing the URLs is fine with me as long as the old urls will work too. >>> >>> But do we need to change the filenames for the blog posts to accomplish >>> that? It's nice that the blog post markdown files start with a date so they >>> naturally sort chronologically. It looks like this hugo PR [1] made it >>> possible to extract date metadata and slug >>> (i.e. dataflow-python-sdk-is-now-public) separately from the filename. >>> >>> [1] https://github.com/gohugoio/hugo/pull/4494 >>> >>> On Thu, Apr 30, 2020 at 10:06 AM Ahmet Altay <al...@google.com> wrote: >>> >>>> >>>> >>>> On Thu, Apr 30, 2020 at 9:55 AM Thomas Weise <t...@apache.org> wrote: >>>> >>>>> For changed URLs, will previous URLs be mapped to avoid broken >>>>> external links? >>>>> >>>> >>>> I believe the answer is yes from Nam's response "For now, we keep the >>>> old URLs working in terms of redirecting them". I very much agree that this >>>> is very important and should work for all existing urls. >>>> >>>> >>>>> >>>>> >>>>> On Thu, Apr 30, 2020 at 9:34 AM Aizhamal Nurmamat kyzy < >>>>> aizha...@apache.org> wrote: >>>>> >>>>>> Hi, >>>>>> >>>>>> To give a little more context regarding the URLs, the date should >>>>>> still appear on the blog post, but not on the URL. >>>>>> For example, we'd have: >>>>>> >>>>>> https://beam.apache.org/beam/python/sdk/2016/02/25/python-sdk-now-public.html >>>>>> become >>>>>> https://beam.apache.org/blog/dataflow-python-sdk-is-now-public/. >>>>>> >>>>> >>>> I am not a content marketer. IMO, this is a good change. In the past, a >>>> few times, we edited dates on posts (e.g. a release date was entered >>>> incorrectly) and we had to either have a mismatch between dates in the url >>>> and the date in the blog, or change the url. This change simplifies, by >>>> having date only in place (in content metadata). >>>> >>>> >>>>> >>>>>> The blog posts would have a small header showing the title, author >>>>>> and publish date. But the URL would not have it. >>>>>> Thoughts? >>>>>> >>>>>> >>>>>> On Thu, Apr 30, 2020 at 9:23 AM Nam Bui <nam....@polidea.com> wrote: >>>>>> >>>>>>> Hi, >>>>>>> >>>>>>> @altay: Hey hey. Yeah, I didn't expect the baseUrl of staging >>>>>>> version is " >>>>>>> http://apache-beam-website-pull-requests.storage.googleapis.com/11554/"; >>>>>>> which also includes "/11554", and Hugo considers it as a path so it >>>>>>> breaks >>>>>>> the path of "static files" (like images). We made a fix. Now I'm >>>>>>> working on >>>>>>> "getting git to recognize files as renames" as you suggested. >>>>>>> >>>>>>> @robert: The dates are nice but it causes verbose/long/ugly URLs. We >>>>>>> discussed with Aizhamal in the development stage and agreed to get rid >>>>>>> of >>>>>>> this. For now, we keep the old URLs working in terms of redirecting >>>>>>> them. >>>>>>> However, from now on, we should change the name convention on blog >>>>>>> posts to >>>>>>> have a fancy URL like "beam.apache.org/blog/myblogpost.md". :) >>>>>>> >>>>>>> >>>>>>> >>>>>>> On Thu, Apr 30, 2020 at 2:57 AM Robert Bradshaw <rober...@google.com> >>>>>>> wrote: >>>>>>> >>>>>>>> On Wed, Apr 29, 2020 at 5:08 PM Ahmet Altay <al...@google.com> >>>>>>>> wrote: >>>>>>>> >>>>>>>>> Nam, this looks better. At least links are working, and the >>>>>>>>> website visually looks similar and generally in good shape. I think >>>>>>>>> there >>>>>>>>> are still issues. For example, I do not see any of the images (e.g. >>>>>>>>> the >>>>>>>>> beam logo on top left is missing.) >>>>>>>>> >>>>>>>>> On Wed, Apr 29, 2020 at 3:11 PM Brian Hulette <bhule...@google.com> >>>>>>>>> wrote: >>>>>>>>> >>>>>>>>>> I left a comment on the PR [1]. I think the reason all of the >>>>>>>>>> website content is not being tracked as file renames is because >>>>>>>>>> there was a >>>>>>>>>> series of commits that created files in the new directory, and then >>>>>>>>>> one >>>>>>>>>> commit that deleted the old directory. If there were a single commit >>>>>>>>>> with >>>>>>>>>> all of the deleted and new files, git would surely recognize they are >>>>>>>>>> effectively renameds and mark them as such. Maybe we just need to >>>>>>>>>> get all >>>>>>>>>> these commits squashed into one? >>>>>>>>>> >>>>>>>>>> [1] >>>>>>>>>> https://github.com/apache/beam/pull/11554#issuecomment-621489844 >>>>>>>>>> >>>>>>>>> >>>>>>>>> Nam, could you try this? If we can get git to recognize these as >>>>>>>>> renames, review process would be much easier. >>>>>>>>> >>>>>>>> >>>>>>>> +1. >>>>>>>> >>>>>>>> Alternatively, create a commit that just moves the files into a new >>>>>>>> location (which git can always detect), then sit the edits on top of >>>>>>>> that >>>>>>>> (which should preserve history better). >>>>>>>> >>>>>>>> Also, is there a reason the dates were removed from the blog post >>>>>>>> filenames? For content like that, the dates are nice. >>>>>>>> >>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>>> >>>>>>>>>> On Wed, Apr 29, 2020 at 10:39 AM Nam Bui <nam....@polidea.com> >>>>>>>>>> wrote: >>>>>>>>>> >>>>>>>>>>> Hi guys, >>>>>>>>>>> >>>>>>>>>>> I'm Nam - from the responsible team of Apache Beam website >>>>>>>>>>> migration. I am pleased to answer some of the questions here. >>>>>>>>>>> >>>>>>>>>>> @aizhamal: Thanks for informing to the community. :) >>>>>>>>>>> @altay, @robertwb: Yes. there is a problem with the staged >>>>>>>>>>> version at the moment. We didn't expect some behaviours on the build >>>>>>>>>>> process. So, we fixed it today and been waiting for @pablo to >>>>>>>>>>> re-run it >>>>>>>>>>> again. The purpose of this PR is to migrate completely Beam site >>>>>>>>>>> from >>>>>>>>>>> Jekyll to Hugo. Therefore, a bunch of deleted markdown files are >>>>>>>>>>> from >>>>>>>>>>> Jekyll which was located at `beam/website/src`, and Hugo is located >>>>>>>>>>> at >>>>>>>>>>> `beam/website/www` now. In `beam/website/README.md`, I wrote down >>>>>>>>>>> about >>>>>>>>>>> running the Hugo website locally, although it is actually same as >>>>>>>>>>> Jekyll >>>>>>>>>>> (because it's also set up with Docker & Gradle). In >>>>>>>>>>> `beam/website/CONTRIBUTE.md`, I guided people on how to get started >>>>>>>>>>> with >>>>>>>>>>> Hugo on the Beam website. There is also a link in the "Translation >>>>>>>>>>> Guide" >>>>>>>>>>> section which points to a branch of multilingual provenance, and it >>>>>>>>>>> will >>>>>>>>>>> become a next PR soon. >>>>>>>>>>> >>>>>>>>>>> Please let me know if you need more details. Feel free to ask >>>>>>>>>>> any questions and I will get back to you with answers. I'm so sorry >>>>>>>>>>> if I >>>>>>>>>>> answer a little bit due to the timezone. :) >>>>>>>>>>> >>>>>>>>>>> Best regards, >>>>>>>>>>> Nam >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> On Tue, Apr 28, 2020 at 8:49 PM Aizhamal Nurmamat kyzy < >>>>>>>>>>> aizha...@apache.org> wrote: >>>>>>>>>>> >>>>>>>>>>>> Adding +Nam Bui <nam....@polidea.com> and +Karolina Rosół >>>>>>>>>>>> <karolina.ro...@polidea.com> to follow up on questions. >>>>>>>>>>>> >>>>>>>>>>>> On Tue, Apr 28, 2020 at 11:34 AM Ahmet Altay <al...@google.com> >>>>>>>>>>>> wrote: >>>>>>>>>>>> >>>>>>>>>>>>> I am having trouble reviewing the staged version. What is the >>>>>>>>>>>>> best way to review this change? >>>>>>>>>>>>> >>>>>>>>>>>>> Do we expect any changes to markdown files, beyond some >>>>>>>>>>>>> metadata? >>>>>>>>>>>>> >>>>>>>>>>>>> On Tue, Apr 28, 2020 at 10:45 AM Robert Bradshaw < >>>>>>>>>>>>> rober...@google.com> wrote: >>>>>>>>>>>>> >>>>>>>>>>>>>> Thanks. It'll be great to better support more languages. >>>>>>>>>>>>>> >>>>>>>>>>>>>> I looked at the PR and there seems to be no >>>>>>>>>>>>>> provenance/history. E.g. all the content seems to be entirely >>>>>>>>>>>>>> new files >>>>>>>>>>>>>> rather than diffs from the old. (There also seems to be a huge >>>>>>>>>>>>>> amount of >>>>>>>>>>>>>> auto-generated js code as well.) >>>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> I agree. This makes it very hard to review. I also see a bunch >>>>>>>>>>>>> of deleted markdown files. Are they not getting migrated? >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> On Tue, Apr 28, 2020 at 10:23 AM Aizhamal Nurmamat kyzy < >>>>>>>>>>>>>> aizha...@apache.org> wrote: >>>>>>>>>>>>>> >>>>>>>>>>>>>>> Hello everybody, >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> We are almost done migrating the Apache Beam website from >>>>>>>>>>>>>>> Jekyll to Hugo. You can see the PR in [1], and we'd love to >>>>>>>>>>>>>>> hear your >>>>>>>>>>>>>>> feedback/comments on the PR. It includes detailed guidelines on >>>>>>>>>>>>>>> contributing to the new Hugo-based website and adding >>>>>>>>>>>>>>> translations to pages >>>>>>>>>>>>>>> [2]. For those who are curious about adding new languages, we >>>>>>>>>>>>>>> will provide >>>>>>>>>>>>>>> a proof of concept in the next couple of days in this thread. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Since we want to move forward with the PR, I would like to >>>>>>>>>>>>>>> ask the community to hold off changes to the current Beam >>>>>>>>>>>>>>> website for a >>>>>>>>>>>>>>> week, until we are able to review and merge the PR. Is this >>>>>>>>>>>>>>> acceptable to >>>>>>>>>>>>>>> everyone? >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> In case anyone missed my previous email with the background >>>>>>>>>>>>>>> for the website migration, you can find more context here [3]. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Thanks, >>>>>>>>>>>>>>> Aizhamal >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> [1] https://github.com/apache/beam/pull/11554 >>>>>>>>>>>>>>> [2] >>>>>>>>>>>>>>> https://github.com/apache/beam/blob/256b7042bf504b94f161ca03b388a2ba247918d9/website/CONTRIBUTE.md >>>>>>>>>>>>>>> [3] >>>>>>>>>>>>>>> https://lists.apache.org/thread.html/r7fa6d710c0a1959cce5108e460d71c306ce5756cf96af818b41cb7ca%40%3Cdev.beam.apache.org%3E >>>>>>>>>>>>>>> >>>>>>>>>>>>>>
