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

[docs/folder-structure] Changed Docs Structure / MDM Fixes #1094

Merged
merged 4 commits into from
Apr 4, 2022

Conversation

hosy
Copy link
Collaborator

@hosy hosy commented Jan 21, 2022

Description

  • added missing link in MDM description
  • moved ios_mdm_tables.adoc output into new folder structure
  • removed no longer needed docs folder

Related Issue

#1070 #1075 #1082

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • Added an issue with details about all relevant changes in the iOS documentation repository.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • Added changelog files for the fixed issues in folder changelog/unreleased

- added missing link in MDM description
- moved ios_mdm_tables.adoc output into new folder structure
- removed no longer needed docs folder
@hosy hosy self-assigned this Jan 21, 2022
@hosy hosy linked an issue Jan 21, 2022 that may be closed by this pull request
@hosy hosy mentioned this pull request Jan 21, 2022
@mmattel
Copy link
Contributor

mmattel commented Jan 21, 2022

Great, many thanks!

Note that I have copied the ios_mdm_tables.adoc file from tag 11.8.0 and 11.7.0 (actual doc versions) which I will include hard into the documentation, while the new file of master and later will be included by link dynamically.

@@ -32,4 +32,4 @@ jobs:
uses: stefanzweifel/git-auto-commit-action@v4
with:
commit_message: Configuration documentation updated
file_pattern: doc/CONFIGURATION.json docs/modules/ROOT/pages/ios_mdm_tables.adoc
file_pattern: doc/CONFIGURATION.json doc/antora/pages/ios_mdm_tables.adoc
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please generate on 2 places:

  • doc/CONFIGURATION.adoc
  • doc/antora/pages/mdm_tables.adoc (wie MMattel will…)

Copy link
Contributor

@mmattel mmattel Jan 25, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A copy (double generation) of mdm_tables.adoc named doc/CONFIGURATION.adoc does not make any sense especially because you want to combine several markdown files into doc/CONFIGURATION.adoc for ios-app internal documentation purposes, see your comment: #1094 (comment). Therefore I guess this was just a mistake 😄

It is btw ok if you really want to use asciidoc (.adoc) to render ios-app internal stuff for documentation purposes, but if you do not need these capabilities, my suggestion is to stay with markdown (.md)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nothing gets combined. Other *.md,s will just be deleted. I need the MDM content accessible in doc/CONFIGURATION.adoc. If you want to have the content in doc/antora/pages/, @hosy will add a copy for your purpose there. up2u…

@michaelstingl
Copy link
Contributor

2 tiny additions:

  • Move screenshots to doc/images
  • enterprise/gomplate/Branding.plist.tmpl ==> move to tools folder

@michaelstingl
Copy link
Contributor

… and can we get rid of these?

  • APP_BUILD_FLAGS.md
  • BUILD_CUSTOMIZATION.md
  • CONFIGURATION.md

(and unite them all in the mighty doc/CONFIGURATION.adoc)

@mmattel
Copy link
Contributor

mmattel commented Jan 25, 2022

@hosy Just found by chance that we need an additional text change to have the correct doc link.

ios-sdk/doc/CONFIGURATION.md:18

The file located at `doc/CONFIGURATION.json` provides a machine-readable list of available settings.
For the `ios-sdk` repository, this includes only settings implemented in the SDK. For the full list of
settings available for configuration, please see
[`docs/modules/ROOT/pages/ios_mdm_tables.adoc`]
(https://github.com/owncloud/ios-app/blob/master/docs/modules/ROOT/pages/ios_mdm_tables.adoc)
in the `ios-app` repository.

- moved gomplate branding template to tools
- deleted no longer needed documentation files (values are available in configuration.adoc)
- renamed files
- deleted antora folder
- added readme file in doc folder
@hosy
Copy link
Collaborator Author

hosy commented Jan 28, 2022

@mmattel @michaelstingl I did further changes in this branch, with the following changes:

  • moved fastlane screenshots into doc folder
  • moved gomplate branding template to tools
  • deleted no longer needed documentation files (values are available in configuration.adoc)
  • renamed files
  • deleted antora folder
  • added readme file in doc folder

Please have a look again. Thank you!

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ hosy
❌ Matthias Hühne


Matthias Hühne seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@hosy
Copy link
Collaborator Author

hosy commented Mar 29, 2022

@michaelstingl merge conflict is solved and you can add your review
@mmattel please add your review too.

@michaelstingl
Copy link
Contributor

@michaelstingl merge conflict is solved and you can add your review

This is un-reviewable. Please merge, then we'll iterate from there…

@mmattel
Copy link
Contributor

mmattel commented Mar 31, 2022

This is un-reviewable. Please merge, then we'll iterate from there…

To proceed, pls approve (2a necessary), I did already.

Copy link
Contributor

@michaelstingl michaelstingl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's try this… 🤞

@hosy hosy merged commit 420e5ff into master Apr 4, 2022
@delete-merged-branch delete-merged-branch bot deleted the docs/folder-structure branch April 4, 2022 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[MDM] move ios_mdm_tables.adoc to doc/CONFIGURATION.adoc
4 participants