-
Notifications
You must be signed in to change notification settings - Fork 163
Migrate owncloud versions and add validation for more elements #156
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
Conversation
…elds prevented apps from being uploaded
| <xs:element name="lib" type="min-max-version" minOccurs="0" | ||
| maxOccurs="unbounded"/> | ||
| <xs:element name="owncloud" type="owncloud" minOccurs="1" | ||
| <xs:element name="nextcloud" type="nextcloud" minOccurs="1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
min === 0 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minimum version is required (also in core afaik)
provided owncloud tags are migrated to nextcloud tags, that way there will always be a nextcloud tag if set properly
…f standalone, shipped or default_enable elements are present
|
I've also added the following things:
|
…ninstall elements, fail validation if remote or public is present
|
More validation, see updated first post |
|
👍 from IRC |
Endpoints are all hardcoded since 7bdde98 and were formally deprecated within info.xml in nextcloud/appstore#156. Follow-up PR will clean-up references elsewhere. Signed-off-by: Josh <[email protected]>
Works in the following way:
If no tag exists the versions will be taken from and following will be done:
I hope that's enough and we dont need to consider further exceptions
Furthermore:
@nickvergessen @janibonnevier @adsworth @LukasReschke @MorrisJobke please review