On Mon, May 4, 2020 at 6:02 PM Thomas Weise <thomas.we...@gmail.com> wrote:
>
> I took a brief look at [1] and looks good overall.
>
> There are some changes missing though (for example [2]), are you planning to 
> add more recent commits later?
>
> Also, there was an earlier question from Brian regarding the possibility to 
> retain the post dates in blog file names. I would second that, it would make 
> the posts significantly easier to navigate.

I'm OK with removing them from the URL if they're to distracting, but
generally agree here. (If it's too difficult, it's not a huge issue.)

> [1] 
> http://apache-beam-website-pull-requests.storage.googleapis.com/11554/index.html
> [2] 
> http://apache-beam-website-pull-requests.storage.googleapis.com/11554/documentation/runners/flink/index.html
>
> Thomas
>
> On Mon, May 4, 2020 at 11:06 AM Hannah Jiang <hannahji...@google.com> wrote:
>>
>> Hi Aizhamal,, yes, Wednesday sounds good to me. Thank you.
>>
>>
>> On Mon, May 4, 2020 at 10:40 AM Aizhamal Nurmamat kyzy <aizha...@apache.org> 
>> wrote:
>>>
>>> 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 and +Karolina Rosół 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
>>
>> --
>> Please help me know how I am doing: go/hannahjiang-feedback

Reply via email to