Skip to content
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

Translation of caching chapter to French #669

Merged
merged 15 commits into from
Mar 6, 2020
Merged

Translation of caching chapter to French #669

merged 15 commits into from
Mar 6, 2020

Conversation

allemas
Copy link
Contributor

@allemas allemas commented Feb 20, 2020

Makes progress on #539

It took a while, but it's finally over. :)
FR Translations of caching chapter and it's open for review.

Hope that master branch synch is ok.

allemas and others added 10 commits February 20, 2020 19:30
# This is the 1st commit message:

update

# The commit message #2 will be skipped:

# review

# The commit message #3 will be skipped:

# review #2

# The commit message #4 will be skipped:

# advance

# The commit message #5 will be skipped:

# update

# The commit message #6 will be skipped:

# update translation

# The commit message #7 will be skipped:

# update

# The commit message #8 will be skipped:

# update
#
# update

# The commit message #9 will be skipped:

# update
(typofixes, translating alternatives)
    To facilitate the review of original text vs. translation side-by-side.

Also: microtypo fixes.
@tunetheweb tunetheweb added the translation world wide web label Feb 20, 2020
@tunetheweb tunetheweb added this to TODO in Web Almanac 2019 via automation Feb 20, 2020
@tunetheweb
Copy link
Member

Thanks @allemas !

Away at the moment so will have a proper look at the weekend, but did spot that it looks like you missed the last set of updates I did to this chapter in a56d6ef as part of #660

Could you look to do the same changes in the French version?

@allemas
Copy link
Contributor Author

allemas commented Feb 20, 2020

Ho ! ok i'll do that :)

@allemas
Copy link
Contributor Author

allemas commented Feb 22, 2020

It's now ok.

I hope :D

Copy link
Member

@tunetheweb tunetheweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spotted a few things @allemas

src/content/fr/2019/caching.md Outdated Show resolved Hide resolved
src/content/fr/2019/caching.md Show resolved Hide resolved
src/content/fr/2019/caching.md Outdated Show resolved Hide resolved
src/content/fr/2019/caching.md Outdated Show resolved Hide resolved
src/content/fr/2019/caching.md Outdated Show resolved Hide resolved
allemas and others added 2 commits February 22, 2020 18:13
Co-Authored-By: Barry Pollard <barry_pollard@hotmail.com>
@tunetheweb
Copy link
Member

Any more thoughts on my three unresolved comments @allemas ?

@borisschapira it looks like you have reviewed this already. Does that mean you're happy to merge it from your side?

Thanks,
Barry

allemas and others added 2 commits March 6, 2020 11:35
If it's not too much effort, could you also fix this in the English version as part of this PR as looks wrong there:

6% of requests have a time to time (TTL)

should be:

6% of requests have a Time to Live (TTL)

Co-Authored-By: Barry Pollard <barry_pollard@hotmail.com>
Do we need to state that all the directives are English language terms or is that overkill? If so need to check this doesn't mess up the markdown->HTML script.

Co-Authored-By: Barry Pollard <barry_pollard@hotmail.com>
@allemas allemas merged commit b19af35 into HTTPArchive:master Mar 6, 2020
Web Almanac 2019 automation moved this from TODO to Done Mar 6, 2020
allemas added a commit that referenced this pull request Mar 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation world wide web
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants