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

165 Add FDC3 Explained to the toolbox folder #535

Merged
merged 11 commits into from
Mar 18, 2022

Conversation

rikoe
Copy link
Contributor

@rikoe rikoe commented Dec 26, 2021

resolves #165

I have recreated the FDC3 Explained pull request based on #367.

I have made the following changes:

  • Rebased on latest master
  • Move from website/static folder to toolbox/fdc3-explained
  • Remove legacy version and demo apps
  • Some renames and cleanup of files
  • Improve handling of switching between different versions with drop-down
  • Ensure that user informed no FDC3 version is detected when not running in a desktop agent
  • Fix handling of OpenFin detection
  • Use fdc3.getInfo for v1.2
  • Add README.md
  • Squashed commits to reduce churn

I believe if I did the build correctly, the deployment to the toolbox folder on the website should now also work, alongside that of FDC3 Workbench.

image

@kriswest I believe this addresses all your issues:

  • Ensure all files encapsulated in a folder under the 'toolbox' folder to ensure we can use it for other tools, i.e. toolbox/fdc3-explained
  • Rename the root page from toolbox/fdc3_explained.html to toolbox/fdc3-explained/index.html (and correct relative URLs in that file).
  • Also move toolbox/fdc3_explained_logo.svg -> toolbox/fdc3-explained/logo.svg
  • Remove duplicate files (including legacy.html)
  • Ensure that the 1.2 version uses the the fdc3.getInfo() function to get provider details - meaning vendor specific code can be removed
  • Remove vendor specific manifest files in toolbox/openfin directory
  • Add a Readme file to the toolbox/fdc3-explained folder

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Dec 26, 2021

CLA Signed

The committers are authorized under a signed CLA.

WatsonCIQ and others added 7 commits December 26, 2021 18:47
Update toolbox/fdc3-explained/index.html

Co-authored-by: Kris West <kris@cosaic.io>
Adding the legacy, pre 1.1 app
- Move from website/static folder to toolbox/fdc3-explained
- Remove legacy.html
- Rename logo svg file
- Remove toolbox/demoapps
- Split out JavaScript and CSS from v1.0 html file, and clean up
- Remove dependency on versions.js file, just hard code to simplify code
- Improve handling of switching between different versions with drop-down
- Ensure that user informed no FDC3 version is detected when not running in a desktop agent
- Fix handling of OpenFin detection
- Use fdc3.getInfo for v1.2
- Add README.md
@kriswest kriswest requested a review from a team January 4, 2022 11:49
Copy link
Contributor

@donbasuno donbasuno left a comment

Choose a reason for hiding this comment

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

Looks good to me - ship it!

…data, issues with join and leave channel buttons, formatting of contexts, separating get context action onto a button for clarity/consistency, adjusting styles
@kriswest
Copy link
Contributor

Awaiting a few tweaks that are on a PR into Riko's branch. Will merge when those have been merged into this PR.

@kriswest kriswest changed the title Add FDC3 Explained to the toolbox folder 165 Add FDC3 Explained to the toolbox folder Mar 11, 2022
@kriswest kriswest added toolbox Apps or code that isn't part of the standard but supports it use website labels Mar 11, 2022
Copy link
Contributor

@kriswest kriswest left a comment

Choose a reason for hiding this comment

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

Awaiting a few tweaks that are on a PR into Riko's branch. Will merge when those have been merged into this PR.

@kriswest kriswest requested a review from mistryvinay March 16, 2022 13:08
Fixing several issues in fdc3 explained
Copy link
Contributor

@mistryvinay mistryvinay left a comment

Choose a reason for hiding this comment

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

This looks good to me. 👍

@kriswest kriswest self-requested a review March 18, 2022 12:32
Copy link
Contributor

@kriswest kriswest left a comment

Choose a reason for hiding this comment

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

LGTM

@kriswest kriswest merged commit f885209 into finos:master Mar 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
toolbox Apps or code that isn't part of the standard but supports it use website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FDC3 Explained - demo app for beginners?
5 participants