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

New certifcate panel on game, new profile list, and admin settings #30

Merged
merged 3 commits into from
Jun 14, 2022

Conversation

sei-jbritton
Copy link
Contributor

@sei-jbritton sei-jbritton commented May 12, 2022

Corresponding API changes: cmu-sei/Gameboard#35

  • New expandable panel to view and print certificate on game info page; only appears if the game has completed, a template is defined, and the player was enrolled
  • New page /profile/certificates to list all completed games with certificates that the user competed in
  • New text area in admin settings for HTML template along with toggle to view info and available dynamic properties available
  • Note: Only supply HTML content that would go inside<body>...</body> in the document. This can include adding styles as inline or internal CSS (e.g. <style>div {color:red;}</style>).
  • The print button opens a new window populated with the generated HTML. It modifies it slightly to support better printing to PDF.

This allows an admin to copy in an HTML template for a game in the settings. Players can view a list of past certificates from their profile page or can view it on the main game page after a game completes. There is also a shortcut to print only the certificate content in a new window.

Copy link
Collaborator

@sei-mkaar sei-mkaar left a comment

Choose a reason for hiding this comment

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

Tested with Foundry Appliance 👍

@sei-mkaar sei-mkaar merged commit e40c36d into main Jun 14, 2022
@sei-mkaar sei-mkaar deleted the certificate-pr branch June 14, 2022 16:09
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