v7.0.3+beta.2 #564
Merged
v7.0.3+beta.2 #564
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Was resulting in a more generic exception being raised
- Stops Kodi unnecessarily trying to load non-existent studio logos
- Avoid clearing caches and resetting cache items on user change
- Make refresh context menu item actually refresh - First page of playlist will be cached for 5 minutes, remaining pages cached for 1 hour - Extract urls from video description cached for 1 day, rather than 1 week - Don't clear function cache on service restart (may cause regressions but cipher/nsig handling is not currently used)
- Update make-release workflow
- Changelog will be used to populate release details
- Changelog entries to follow Github markdown formatting rules
- addon.xml updates will preserve whitespace� - news element in addon.xml will be populated from changelog without issue references
- Updated submit-release workflow
- addon.xml updates will preserve whitespace
- news element in addon.xml will be populated from changelog without issue references
- news is also used for repository PRs as part of submit-release workflow� - Github references either need to be removed or updated. Removed for simplicity
- Was failing due to empty news element. Remove element and add in release workflows
|
@MoojMidge |
|
This is just a Pull Request. Once the changes are merged, then the updated installation files will be created as part of making a new release. Not planning any further changes to v7.0.3+beta.2, so this can be merged, however there may be some changes to the associated unofficial version to allow for customisation of the sort order display, to address #559. If it turns out the changes for the sort order customisation are too big, may just leave that along with any other updates, for a future release. |
- Also ensure category label is updated when name is updated
- Params passed as a parameter to constructor were not being included in generated uri
- Must be contained in directory uri to be parsed in new Context params
|
Unofficial changes are too extensive and require further testing, merging as-is to resolve v7.0.3+beta.1 bugs. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changed
Fixed