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

[stable27] Revert "perf(dashboard): implement widget item api v2" #668

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

st3iny
Copy link
Member

@st3iny st3iny commented Oct 30, 2023

Feel free to close if this backport is not required.

Manual backport of #667

This reverts commit f890baf.

I tested this locally and the dashboard widgets works like it did before (builtin viewer etc.).

@st3iny st3iny added this to the Nextcloud 27.1.4 milestone Oct 30, 2023
@st3iny st3iny self-assigned this Oct 30, 2023
@blizzz
Copy link
Member

blizzz commented Nov 15, 2023

some tests are failing and also requires a rebuild

@blizzz
Copy link
Member

blizzz commented Nov 16, 2023

moving to 27.1.5

@blizzz
Copy link
Member

blizzz commented Dec 4, 2023

/compile /

@blizzz
Copy link
Member

blizzz commented Dec 5, 2023

@st3iny conflicts

@blizzz
Copy link
Member

blizzz commented Dec 7, 2023

moving to 27.1.6

@blizzz
Copy link
Member

blizzz commented Jan 15, 2024

@Pytal @ChristophWurst what to do about this PR?

@Pytal
Copy link
Member

Pytal commented Jan 16, 2024

@Pytal @ChristophWurst what to do about this PR?

Not sure as the original PR has not been merged either #667, would say @st3iny knows best :)

This reverts commit f890baf.

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
@st3iny st3iny force-pushed the revert/644-stable27 branch from 053e1b1 to ebdbc7b Compare January 17, 2024 09:56
@st3iny st3iny merged commit 41e13f7 into stable27 Jan 17, 2024
18 checks passed
@st3iny st3iny deleted the revert/644-stable27 branch January 17, 2024 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants