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

Created list to filter unique events #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BjornGrylls
Copy link

Tilføjede et tjek så events over flere dage kun bliver tilføjet én gang.
Løsning til issue #28
Dette er ikke testet.

@soeren-b-c
Copy link
Owner

@BjornGrylls Mange tak for dette pull request!

Jeg har ikke tid til at kigge på det lige nu, men jeg vender tilbage enten med kommentarer eller Merge på et senere tidspunkt.

@soeren-b-c
Copy link
Owner

soeren-b-c commented Jun 4, 2023

@BjornGrylls Hej igen. Undskyld det har taget så længe, jeg har haft travlt.

Jeg kan desværre ikke få din kode til at virke.

Jeg får blot en tom kalender fil, og følgende i min log: regExp.exec is not a function

Hvis jeg ændrer

              var regExp = '/absid=(\\d+)/'; // Create regex

til

              var regExp = /absid=(\\d+)/; // Create regex

forsvinder den første fejl, men jeg får Cannot read properties of null (reading '1'), og stadig intet indhold i .ics filen.

Jeg har desværre dårligt tid, og meget sløje node skills, så jeg gør nok ikke meget mere for at få denne kode til at virke. Men hvis du får det fix'et kigger jeg gerne igen, og tester igen.

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.

2 participants