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

feat: show error message from opml import in web-ui #3036

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

wofferl
Copy link
Collaborator

@wofferl wofferl commented Jan 8, 2025

Summary

Return opml import error message as api response to show it in the web-ui.

Checklist

@wofferl wofferl force-pushed the opml_import_dialog branch from ca99a17 to 75301cb Compare January 8, 2025 21:44
@wofferl wofferl force-pushed the opml_import_dialog branch from 75301cb to bce6d63 Compare January 8, 2025 21:46
@wofferl wofferl added API Impact API/Backend code frontend impact Javascript/Frontend code labels Jan 8, 2025
@Grotax Grotax merged commit 6dbfa9a into nextcloud:master Jan 9, 2025
26 checks passed
Grotax added a commit that referenced this pull request Jan 10, 2025
Changed
- add explanations for the individual values in the feed information table (#3031)
- show error message from `opml` import in web-ui (#3036)
- disable new setting "nextUpdateTime" introduced in #2999 by default (#3039)

Fixed
- fix proxy port removed if standard port for the protocol (#3027)
- background updater may stumble over invalid datetime strings from feeds (#3028)

Signed-off-by: Benjamin Brahmer <[email protected]>
@Grotax Grotax mentioned this pull request Jan 10, 2025
Grotax added a commit that referenced this pull request Jan 10, 2025
Changed
- add explanations for the individual values in the feed information table (#3031)
- show error message from `opml` import in web-ui (#3036)
- disable new setting "nextUpdateTime" introduced in #2999 by default (#3039)

Fixed
- fix proxy port removed if standard port for the protocol (#3027)
- background updater may stumble over invalid datetime strings from feeds (#3028)

Signed-off-by: Benjamin Brahmer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Impact API/Backend code frontend impact Javascript/Frontend code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants