Hey,
it would be great if the project contains a typescript index.d.ts file. So in typescript projects as in Angular the compiler doesn't claim that the function isHoliday doesn't exist.

Hi @DaniSchenk,

erstmal vielen Dank, dass du dir die Mühe gemacht hast ein Plugin für die deutschen Feiertage zu schreiben und es dann der Community zur Verfügung stellst :)

Da du ja selbst aus München kommst, bist du bestimmt selbst schon über den Fall gestolpert, dass du Maria Himmelfahrt gerne als Feiertag betrachten möchtest da er fast in ganz Bayern ein Feiertag ist.

Aus deiner Dokumentation
const isHolidayInBavaria = moment('2017-08-15').isHoliday('BY'); // returns false - is not a holiday in BY

Gibt es eine Methode mit der sich dieses Verhalten steuern lässt oder wie handhabst du solche Fälle.

Danke für deine Zeit.

Liebe Grüße

Tobi

A holiday should be recognized on the whole day.
The example snippet const nowIsHoliday = moment().isHoliday() in the README suggests, that this is also meant to be in the feature scope.

Example Code:

const moment = require('moment-feiertage')

console.log(moment('2020-12-25 17:00').isHoliday()) // false
console.log(moment('2020-12-25 00:00').isHoliday()) // 1. Weihnachtsfeiertag

A possible fix would probably be, to change

if (_moment.isSame(holidays[h].date, 'day')) { 

Das ist glaube cih nciht drinne.

Feiertage Deutschland » Reformationstag
Reformationstag in Deutschland
Achtung: Anlässlich des 500jährigen Jubiläums von Martin Luthers Thesenanschlag: Der Reformationstag wird einmalig am 31.10.2017 ein bundesweiter Feiertag sein.

hey thank you for that plugin.

You merged the commit for console.log but don't published a new version to npm. So I still get huge console.logs wen using your plugin.

Can you please update the version in your package.json and make npm publish again?

Hi, I use your plugin and it's very useful for me.
But now I have to update moment to version 2.29.
Will you update your peerDependency in the next time?
Best regards,
Peter

Originally posted by @pit999 in #28 (comment)

On react-native isHoliday() is always undefined. Any idea how to add RN support?

If you use the Google Places API, for the query of the federal states to determine the holidays, then you have to remap the value for Lower Saxony (NDS) to the plugin compatible value (NI).

So it would be a good idea to use the values that Google uses for administrative_area_level_1 as short_name also as check for the federal states

MV decided to celebrate the Internationaler Frauentag (8th of march) from 2023 on

Hallo, mir ist per Zufall dieses Fehler aufgefallen:

TypeError: Cannot read property 'length' of undefined
at _isHoliday106 (%PROJECT%\node_modules\moment-feiertage\build\index.js:53:39)
at _isHoliday (%PROJECT%\node_modules\moment-feiertage\build\index.js:85:23)

moment: ^2.29.1
moment-duration-format: ^2.3.2
moment-feiertage: ^2.0.3
var holiday = moment().isHoliday();

Ich bin den Part mit dem Debugger durchgegangen und dabei auf eine for-in-Schleife gestoßen. Problem ist, dass diese ebenfalls durch die Prototypen itteriert und dadurch Attribute verarbeitet, die keine Feiertage beinhalten:

Ich habe hier zwei Vorschläge mit denen man das Problem lösen könnte.

1. via Object.keys und for-of:

for (var h of Object.keys(holidays)) {
    if (_moment.isSame(holidays[h].date, 'day')) {
        if (holidays[h].state.length === 0) {
            return h;
        }
        else {
            if (_state && holidays[h].state.indexOf(_state) > -1) {
                return h;
            }
            else {
                return false;
            }
        }
    }
}

2. via Object.entries und for-of

for(var [h, data] of Object.entries(holidays)) {
    if (_moment.isSame(data.date, 'day')) {
        if (data.state.length === 0) {
            return h;
        }
        else {
            if (_state && data.state.indexOf(_state) > -1) {
                return h;
            }
            else {
                return false;
            }
        }
    }
}

Mit diesen Änderungen funktionierte das Modul bei mir wieder.
- ggf. erstelle ich auch gerne einen Pull Request

Viele Grüße
ACertainCoder