Skip to content

RRule logic to slicken human-readable strings #239

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.

Sign up for GitHub

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

Jump to bottom
Draft
jkrrv wants to merge 441 commits into
base: next
Choose a base branch
from
Draft

RRule logic to slicken human-readable strings #239

jkrrv wants to merge 441 commits into from

Conversation

@jkrrv
Copy link
Member

@jkrrv jkrrv commented Dec 8, 2025

No description provided.

jkrrv added 30 commits September 12, 2024 21:30
@jkrrv
Resolving an issue with calendar navigation.
0131cb7
@jkrrv
adding inline report parameter
1e882b3
@jkrrv
Resolve an issue with DST starting and ending. Closes #207.
2691ddc
@jkrrv
Version bump.
60e2832
@jkrrv
Adding ipapi key option, cleanup functions and related items.
28fb01f 
Also, version bump.
@jkrrv
WordPress tested version bump
8864173
@jkrrv
Fix cal grid incrementing issue.
0499005 
Closes #207 again.
@jkrrv
Only show some registration buttons for meetings if they're relevant …
631c7bb 
…in time. Closes #205
@jkrrv
Improving a label.
8ea9692
@jkrrv
docs update
5de3f0f
@jkrrv
Rename process in build yml
d4202d1
@jkrrv
Merge branch 'master' into events
cf4aaca 
# Conflicts:
#	composer.json
#	i18n/TouchPoint-WP.pot
#	package.json
#	src/TouchPoint-WP/Involvement.php
#	src/TouchPoint-WP/Report.php
#	src/TouchPoint-WP/TouchPointWP.php
#	src/python/WebApi.py
#	src/templates/involvement-single.php
#	touchpoint-wp.php
@jkrrv
Removing noreturn that probably never should have been committed.
97e748b
@jkrrv
Starting #209.
26a61c8
@jkrrv
Progress on #209.
cb38382
@jkrrv
Moving updatedDT to server-side only, where it belongs.
21d9ae6
@jkrrv
remove explicit server error
393674e
@jkrrv
Resolve issue where destruct is getting called to late for saves by s…
0597e00 
…aving manually in each location.

Also making siteId truly global across sites.
@jkrrv
Add setting to allow site owners to determine whether their site/chur…
ef1131b 
…ch is using the plugin. Also i18n.
@jkrrv
make list publicly an int
bcdb84b
@jkrrv
Fix a semantic error with jsonSerialize
3ec3165
@jkrrv
Updating readme about Events and stats
7aa2494
@jkrrv
Cron to submit stats. Theoretically closes #209
701eb35
@jkrrv
Correct missing references
e9a5ee5
@jkrrv
Add ability to limit involvement import to a given campus. Closes #208
c8c3d8a
@jkrrv
Automatically update private key periodically.
38e79b5
@jkrrv
Version bump and translation updates
99362cd
@jkrrv
Add a dashboard widget with some simple stats.
f039fa6
@jkrrv
Adding reports to submitted stats
b94429b
@jkrrv
Updating several settings
b78a81e
jkrrv and others added 25 commits November 13, 2025 17:23
@jkrrv
Revert "adding composer.lock and package-lock"
7101b28 
This reverts commit 912b073.
@jkrrv
allow low-risk security vulnerabilities for testing process only
b2591c8
@jkrrv
allow low-risk security vulnerabilities for testing process only
133059b
@jkrrv
Add comprehensive unit tests for Utilities namespace (DateFormats, Da…
585b241 
…teTimeExtended, Http)

Co-authored-by: jkrrv <1238391+jkrrv@users.noreply.github.com>
@jkrrv
Merge branch 'block2' into copilot/add-unit-tests
44386ec
@jkrrv
edits
74ed19d
@jkrrv
Cleaning up tests and moving some things to specialized classes
d78a368
@jkrrv
Add some of the newer classes
94c678e
@jkrrv
Adding Colors tests
8baca14
@jkrrv
Replace Brain Monkey with WordPress filter testing approach and add C…
e0b521a 
…olors filter integration tests

Co-authored-by: jkrrv <1238391+jkrrv@users.noreply.github.com>
@jkrrv
Clean up dependencies and update documentation for WordPress filter t…
67f26df 
…esting

Co-authored-by: jkrrv <1238391+jkrrv@users.noreply.github.com>
@jkrrv
Fix WordPress function definitions in bootstrap - add missing WP func…
9088687 
…tions and remove old Brain Monkey tests

Co-authored-by: jkrrv <1238391+jkrrv@users.noreply.github.com>
@jkrrv
adjusting naming conventions
f0e7027
@jkrrv
preventing an inplicit type conversion
c2a4d36
@jkrrv
Some formatting changes
b250df9
@jkrrv
Add preview-sync endpoint
a987b82
@jkrrv
Resolving a permissions issue that prevents non-admins from applying …
0a56fb7 
…changes
@jkrrv
Merge R&R changes from other branch
d8892f2
@jkrrv
sync and build
b3318aa
@jkrrv
merge tests
b24055c
@jkrrv
line seps
518b8f5
@jkrrv
Checking that composer can be depended on reliably.
cd27b0e
@jkrrv
Starting with RRule library and some tests for handling merges.
4a1a5d9
@jkrrv
updating tests to the API I actually want
262be3f
@jkrrv
Remove incorrect logic so it's just a stub
3427a8c
@jkrrv jkrrv changed the base branch from master to next December 8, 2025 16:56
@jkrrv
Copy link
Member Author

jkrrv commented Dec 8, 2025

@copilot Please add logic for the ScheduleSet:mergeIfPossible() method to combine RRules when possible. When finished, all existing tests should pass. Do not modify any tests.

Copilot AI mentioned this pull request Dec 8, 2025
Draft
Copy link

Copilot AI commented Dec 8, 2025

@jkrrv I've opened a new pull request, #240, to work on those changes. Once the pull request is ready, I'll request review from you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jkrrv