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

Update recommended apps for 25 #739

Merged
merged 1 commit into from
Sep 16, 2022
Merged

Update recommended apps for 25 #739

merged 1 commit into from
Sep 16, 2022

Conversation

PVince81
Copy link
Member

Remove "notes".
Add "recognize".
Add "forms".

Not sure how to test, I tried delete from oc_preferences where appid='firstrunwizard'; but despite page reload the notifications don't appear.

@PVince81 PVince81 added this to the Nextcloud 25 milestone Sep 14, 2022
@PVince81 PVince81 self-assigned this Sep 14, 2022
@PVince81 PVince81 changed the title Update recommended apps Update recommended apps for 25 Sep 14, 2022
@nickvergessen
Copy link
Member

https://github.com/nextcloud/firstrunwizard/blob/master/lib/Listener/BeforeTemplateRenderedListener.php#L80

Manually add the background job or set the config to 1?🤔

@PVince81
Copy link
Member Author

PVince81 commented Sep 15, 2022

Remove "notes".
Add "recognize".
Add "forms".

Signed-off-by: Vincent Petry <vincent@nextcloud.com>
@PVince81 PVince81 force-pushed the update-recommended-apps branch from 108008f to b4f2031 Compare September 16, 2022 08:34
@PVince81
Copy link
Member Author

image

to test:
occ user:setting admin firstrunwizard show 1

please review

@PVince81 PVince81 merged commit e923da8 into master Sep 16, 2022
@PVince81 PVince81 deleted the update-recommended-apps branch September 16, 2022 12:47
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