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

romcal 2.0.0 βeta #250

Merged
merged 341 commits into from
May 18, 2021
Merged

romcal 2.0.0 βeta #250

merged 341 commits into from
May 18, 2021

Conversation

emagnier
Copy link
Collaborator

@emagnier emagnier commented Nov 26, 2020

This PR contain the rewrite of all the codebase, converted to TypeScript. So this is a major pull request!

🚀 A way to test it, is to install this branch (just rebased from dev) and play with romcal. But we will take the required time on the dev branch to test this new beta release, and do everything that need to be done before a final release.

📚 Please also have a look to the new README.md.


This major code rewrite was also a good opportunity to bring new features, refactor the romcal API, and address a few things:

A big thank you @pejulian for daring this big codebase rewrite and refactor on TypeScript, and for all your help and support! And @tukusejssirs for the massive work you make on calendar and localisation data, proofing and making them always better, and for all your vision and suggestions to this project, all of that is invaluable!
This is another big step towards to make romcal 2.0 happen! But there is still work to do 😉

When this PR will be merged in dev, next PRs will be lighter and easier to review. This will also help future contributors to onboard on this project. Here it was an exception due to the change to TypeScript, and all feature and API refactor it bring.

romcalproject and others added 30 commits February 28, 2020 06:02
typedoc only runs on core branches
tukusejssirs and others added 2 commits May 18, 2021 17:09
…21 (#261) & other l10n improvements

* Implement changes in the GRC based on the decreed from 2 Feb 2021

- Change `martha_of_bethany` to `martha_of_bethany_mary_of_bethany_and_lazarus_of_bethany` (see [decree B0066](https://press.vatican.va/content/salastampa/it/bollettino/pubblico/2021/02/02/0066/00135.html));
- Add the following three celebrations to the GRC (see [decree B0065](https://press.vatican.va/content/salastampa/it/bollettino/pubblico/2021/02/02/0065/00136.html)):
	- `gregory_of_narek_abbot`;
	- `hildegard_of_bingen_abbess`;
	- `john_of_avila_priest`;
- Add localised celebration name of `gregory_of_narek_abbot` to all supported locales;
- `sk`: Fix localisation of `hildegard_of_bingen_abbess`;
- `pt-br`: Add localised keys for all three celebrations added to the GRC.

* Fix `martha_of_bethany_mary_of_bethany_and_lazarus_of_bethany` in `en.ts`.

* Fix the title issue of `john_of_avila_priest`
@emagnier
Copy link
Collaborator Author

I think you should upstream features from your (private) repos / romcal versions. You once told me that diocesan calendars are implemented in your downstream code. 😉

My private fork is now quite diverging from this codebase. But it allowed me to address quickly some requirements for a project I was working on. This was also a good place to experiments new features, that I think I need to polish or redesign before adding them here 😉

jarosz and others added 2 commits May 18, 2021 17:18
General names.
Fix TODOs - in Polish primary patron is called 'główny' and secondary has no extra word.
Lowercase bishop title.
Re-translate and lowercase priest title.
Lowercase martyr title.
Commas for companions.
Commas for patrons.
Some other typos.

Add missing translations for keys from calendars/poland.js:
* joseph_sebastian_pelczar_bishop
* our_lady_queen_of_poland
* our_lady_help_of_christians
* hedwig_of_poland
* dedication_of_consecrated_churches
@emagnier emagnier merged commit e5c4ac7 into dev May 18, 2021
@emagnier emagnier deleted the _F-TypeScript branch May 18, 2021 15:47
@emagnier
Copy link
Collaborator Author

BOOM! 🎆

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

Successfully merging this pull request may close these issues.

7 participants