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

[Migration] i18n_subsites #1349

Closed
adiroiban opened this issue Sep 26, 2021 · 18 comments
Closed

[Migration] i18n_subsites #1349

adiroiban opened this issue Sep 26, 2021 · 18 comments

Comments

@adiroiban
Copy link

It would be nice to have the i18n_subsites plugin move it's own repo.

I can help with the migration.

When doing the migration please also consider moving the issues.

Not sure what to do with the existing pull requests

I some PR for i18n are great contribution, yet they were not approved (since 2017)[https://github.com//pull/841]

Thanks!

@georgkrause
Copy link

I'd like to get involved here, too. How can I help?

@haplo
Copy link

haplo commented Dec 2, 2021

I use the i18n_subsites plugin for my blog and would like to help with the migration. Can anybody chime in about what work needs to be done? @justinmayer?

@georgkrause
Copy link

Its all documented here: https://blog.getpelican.com/namespace-plugin-migration.html

@haplo
Copy link

haplo commented Dec 2, 2021

Thank you @georgkrause. So from https://blog.getpelican.com/namespace-plugin-migration.html it looks like we stand at the very beginning:

Migration from Monolithic Repository

Create an issue at the legacy monolithic Pelican Plugins repository and ask a maintainer to create a corresponding new repository under the new Pelican Plugins organization and invite you to join it.

If one of the maintainers could create a reop for i18n_subsites under the pelican-plugins organization then we could get started. Until then I think I'm going to downgrade my Pelican version.

@justinmayer
Copy link
Member

I am swamped at the moment but will endeavor to kick-start the migration process sometime this month. Thanks to everyone for their patience and understanding!

@avaris
Copy link
Member

avaris commented Dec 2, 2021

Until then I think I'm going to downgrade my Pelican version.

But why? pelican is keeping backwards compatibility with old style plugins for a while.

@haplo
Copy link

haplo commented Dec 3, 2021

I am swamped at the moment but will endeavor to kick-start the migration process sometime this month. Thanks to everyone for their patience and understanding!

Thank you for getting back to us @justinmayer, just let us know when it is ready.

Until then I think I'm going to downgrade my Pelican version.

But why? pelican is keeping backwards compatibility with old style plugins for a while.

Upgrading pelican broke my site so I assumed that was the issue, but maybe it's something else. I will debug it, thank you for the heads up @avaris.

@georgkrause
Copy link

@justinmayer Any news about this?

@georgkrause
Copy link

I filtered the subdirectory in its own repository: https://github.com/georgkrause/i18n_subsites

I am willing to transfer the project, just let me know!

@georgkrause
Copy link

As far as I can tell there are 6 open merge requests regarding i18n_subsites, I will try to review them in the next days and merge them into my fork.

@adiroiban
Copy link
Author

@georgkrause thanks. feel free to ping me if you need any help with testing. I don't have much free time, and I might already used a very customized pelican to get my i18n site working, but I will try to help.

@georgkrause
Copy link

@adiroiban Jeah for me its almost the same, which is why I want to cleanup the situation. I cannot stand pulling a huge set of submodules and patching them to make it work any longer. It would be nice if you would use my fork as well, what needs to be merged to make this possible for you? So I can start with reviewing this first.

@adiroiban
Copy link
Author

Thanks George for the work.
I don't remember what part was not working. I looked into this 6 month ago, and I don't know when I will have time to look into this again.

But definitively, next time I will work on the project in which pelican i18n is used, I will switch to use your fork... that I hope will grow into de-facto plugin.

I did a quick copy/paste for my fork

https://gist.github.com/adiroiban/2019f82cc6b6a335ffdd93a06e7143cc

@georgkrause
Copy link

Do I get this right? This is a dump of what you have running in production? This helps a lot, lets integrate this :)

@adiroiban
Copy link
Author

I think there are 2 main changes in my fork.

First is this change for which I have already created a PR #1350

The second one, is support for gettext inside the main SITE dict... but maybe I am wrong and this is already standard functionality.

SITE = {
    'author': _('A Team'),
    'title': _('Delivering great result each time.'),
    'description': _(
        'The A Team works all day and all night long for nothing.'
    ),
}

@rschiang
Copy link

rschiang commented Aug 10, 2022

@adiroiban @georgkrause @haplo
So I happened to merge some of the more uncontroversial PRs (#1299, #1347, #1350), cleaned up and migrated the whole repo, and published a pre-alpha package on PyPI:

https://pypi.org/project/pelican-i18n-subsites

The project repository currently sits here: https://github.com/rschiang/pelican-i18n-subsites. Could you help verify if it works as expected, so we can ask @justinmayer to move it into @pelican-plugins and make us maintainers? 👀

@adiroiban
Copy link
Author

Hi. Sorry for the delay.
I don't have time now to look into this... I did the website changes a long time ago as an experiment.

At work, we don't yet have a multi-lingual website, it was more of an experiment,

Once this goes into production I will check your repo. I started it and I hope I will not forget about it :)

Thanks for putting this all together.

@adiroiban
Copy link
Author

Let me close this as all work should be done in the separate repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants