Skip to content

Navigation Menu

Sign in
Sign up

feat: add last modified information to response - #1844

Open
blizzz wants to merge 3 commits into
main from
poc/noid/last-modified
Open

feat: add last modified information to response #1844
blizzz wants to merge 3 commits into
main from
poc/noid/last-modified

Conversation

@blizzz

@blizzz blizzz commented Jun 19, 2025
edited by enjeck
Loading

Copy link
Copy Markdown
Member

NotModifiedMiddleware respects last modified information stored with a data response. The business logic still runs, but at least clients will not need to do any heavy lifting, as only 304 is returned if the condition pertains.

@blizzz blizzz added enhancement New feature or request 2. developing Work in progress labels Jun 19, 2025
Comment thread lib/Controller/Api1Controller.php Outdated
Comment thread lib/Controller/Api1Controller.php Outdated
enjeck force-pushed the poc/noid/last-modified branch from 0af84d9 to 458b886 Compare October 30, 2025 09:01
@enjeck enjeck self-assigned this Oct 30, 2025

@enjeck enjeck left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm gonna take over this

enjeck force-pushed the poc/noid/last-modified branch from 458b886 to e47afe5 Compare August 29, 2026 23:57
blizzz and others added 3 commits August 30, 2026 09:19
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Enjeck C. <patrathewhiz@gmail.com>
Signed-off-by: Enjeck C. <patrathewhiz@gmail.com>
enjeck force-pushed the poc/noid/last-modified branch from 4c21794 to 54c87d9 Compare August 30, 2026 08:19
enjeck marked this pull request as ready for review August 30, 2026 08:47
@enjeck enjeck added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@juliusknorr juliusknorr juliusknorr left review comments
@enjeck enjeck enjeck left review comments

At least 1 approving review is required to merge this pull request.

Labels

3. to review Waiting for reviews enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /