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

fix(Public-Sharing): Show footer with server details and eventual ToS through PublicTemplateResponse #5184

Merged
merged 1 commit into from
May 2, 2023

Conversation

tcitworld
Copy link
Member

The public calendar view used 'base' Template responses, preventing the footer that's being used on other public sharing pages from showing. This makes use of PublicTemplateResponse to show it automatically. It's however hidden (unchanged) on embed view.

Reference: https://help.nextcloud.com/t/feature-request-impressum-and-privacy-declaration-links/161162

localhost_8080_index php_apps_calendar_p_z9gd4FP8sgB8k8Cs_dayGridMonth_now

@codecov
Copy link

codecov bot commented May 2, 2023

Codecov Report

Patch coverage has no change and project coverage change: +41.51 🎉

Comparison is base (0f64db2) 22.42% compared to head (3a42cf6) 63.94%.

Additional details and impacted files
@@              Coverage Diff              @@
##               main    #5184       +/-   ##
=============================================
+ Coverage     22.42%   63.94%   +41.51%     
  Complexity      373      373               
=============================================
  Files           237       37      -200     
  Lines         11570     1972     -9598     
  Branches       2174        0     -2174     
=============================================
- Hits           2595     1261     -1334     
+ Misses         8975      711     -8264     
Flag Coverage Δ
javascript ?
php 63.94% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 201 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@st3iny st3iny left a comment

Choose a reason for hiding this comment

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

Tested and works.

@st3iny st3iny added this to the v4.4.0 milestone May 2, 2023
@st3iny st3iny added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels May 2, 2023
@st3iny
Copy link
Member

st3iny commented May 2, 2023

/backport to stable4.3

@backportbot-nextcloud backportbot-nextcloud bot added the backport-request A backport was requested for this pull request label May 2, 2023
@st3iny
Copy link
Member

st3iny commented May 2, 2023

/backport to stable3.5

@st3iny
Copy link
Member

st3iny commented May 2, 2023

@tcitworld Please fix php-cs issues: https://github.com/nextcloud/calendar/actions/runs/4859057851/jobs/8661205778?pr=5184

… through PublicTemplateResponse

The public calendar view used 'base' Template responses, preventing the footer that's being used on other public sharing pages from showing. This makes use of PublicTemplateResponse to show it automatically. It's however hidden on embed view.

Reference: https://help.nextcloud.com/t/feature-request-impressum-and-privacy-declaration-links/161162
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
@st3iny st3iny enabled auto-merge May 2, 2023 09:32
@st3iny st3iny merged commit 4d702c1 into main May 2, 2023
26 checks passed
@st3iny st3iny deleted the public-page-for-calendar branch May 2, 2023 09:34
@backportbot-nextcloud backportbot-nextcloud bot removed the backport-request A backport was requested for this pull request label May 2, 2023
tcitworld added a commit that referenced this pull request May 22, 2023
Regression from #5184

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
backportbot-nextcloud bot pushed a commit that referenced this pull request May 25, 2023
Regression from #5184

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
backportbot-nextcloud bot pushed a commit that referenced this pull request May 25, 2023
Regression from #5184

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug Feature: Public sharing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants