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

First stab documentation for Communicator5. Fixes #120. Includes demo pageset fil… #118

Merged
merged 2 commits into from
Jul 24, 2022

Conversation

joedevsys
Copy link
Contributor

…e.(Grid 3 gridset file renamed to identify it as Grid3).

I'm not sure how you want to arrange the links from the guides to the demo files in resources.

@joedevsys joedevsys changed the title First stab documentation for Communicator5 including demo pageset fil… First stab documentation for Communicator5. Fixes #120. Includes demo pageset fil… Jul 23, 2022
@willwade
Copy link
Contributor

Amazing. Thanks Joe. Yeah I'm not sure where we should pop those files. A few different ideas:

  • In the repo. I'd rather not - they can be massive and are zipped
  • On the app's own file sharing sites. e.g. Pagesetcentral or Grids online. But can be a right pain in updating them on there - at least instally when a ton of changes happening
  • I could use a Sharepoint folder from work - I'd give known folk access.

Maybe its the last one and in the long run we go with pagesetcentral etc

If you prefer I'll give you access to our gitbook site if I can Joe. Might make this a marginally easier process!

@willwade
Copy link
Contributor

im an idiot - I failed to see you've put it in the repo. Joe - im going to merge this and pick up the tweaks from gitbook - you can carry on doing changes directly in the markdown - just do another PR when you want - or do via gitbook. Im easy

@willwade willwade merged commit 55ea53e into AceCentre:master Jul 24, 2022
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.

2 participants