Skip to content

Use core function#29

Draft
Jade-GG wants to merge 5 commits into
feature/statamic-6from
feature/use-core-mapping-function
Draft

Use core function#29
Jade-GG wants to merge 5 commits into
feature/statamic-6from
feature/use-core-mapping-function

Conversation

@Jade-GG
Copy link
Copy Markdown
Collaborator

@Jade-GG Jade-GG commented Mar 13, 2026

ref: RAP-1830

Note that this change requires Rapidez v5.

@BobWez98
Copy link
Copy Markdown
Member

maybe retarget to feature/statamic-6 ?

@Jade-GG Jade-GG changed the base branch from master to feature/statamic-6 March 16, 2026 10:50
@Jade-GG
Copy link
Copy Markdown
Collaborator Author

Jade-GG commented Mar 16, 2026

Also now contains #28 because apparently that's not in the statamic 6 branch yet 👀

@royduin
Copy link
Copy Markdown
Member

royduin commented Mar 17, 2026

So this is Rapidez v5 only? And what about #28? That one is merged and for v4?

@Jade-GG
Copy link
Copy Markdown
Collaborator Author

Jade-GG commented Mar 18, 2026

So this is Rapidez v5 only? And what about #28? That one is merged and for v4?

Yes, this new functionality only exists in v5.

How to deal with #28 is actually a tricky question here. Should we revert that in the v4 branch and require upgrading to Statamic 6 to always be combined with Rapidez v5? Or do we make an in-between version that supports v5? My own take would be to do the former and revert it after this branch is merged.

Copy link
Copy Markdown
Member

@indykoning indykoning left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Conflicts

@Jade-GG Jade-GG marked this pull request as draft April 21, 2026 09:57
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.

4 participants