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

Implemented scheduled notifications for configuring Rally time reminders #14

Merged

Conversation

tbm0115
Copy link
Member

@tbm0115 tbm0115 commented Dec 1, 2021

  • Added Rally reminder from NotificationTrigger to remind users of upcoming rally events in-game
    • Added ability to review all upcoming rallies from UserServer view
    • Added ability to copy a share link for a Rally reminder
    • Added ability to add a shared Rally reminder from a link
  • Basic styling cleanup
    • Formatted UserServer information to match presentation style of reminders
    • Added horizontal rules to add better separation between sections
    • Added help text to forms
    • Added descriptions to sections

@tbm0115 tbm0115 self-assigned this Dec 1, 2021
@tbm0115 tbm0115 added enhancement New feature or request lotr-risetowar Related to the LOTR: Rise to War subsite subsite Flags as related to a subsite labels Dec 1, 2021
@tbm0115 tbm0115 merged commit 31996e4 into subsites/LOTR_RiseToWar Dec 1, 2021
@tbm0115 tbm0115 deleted the tbm0115/LOTR_RiseToWar/RallyReminders branch December 1, 2021 09:38
tbm0115 added a commit that referenced this pull request Dec 1, 2021
* Implemented scheduled notifications for configuring Rally time reminders (#14)

* Added help description for NotificationTriggers (#17)

Added help description in the event that the user's browser does not support Notification Triggers. The description contains a link to the new help page.
tbm0115 added a commit that referenced this pull request Dec 4, 2021
* Implemented scheduled notifications for configuring Rally time reminders (#14)

* Added help description for NotificationTriggers (#17)

Added help description in the event that the user's browser does not support Notification Triggers. The description contains a link to the new help page.

* Added Models based on tasks in project. (#35)
tbm0115 added a commit that referenced this pull request Dec 11, 2021
* Implemented scheduled notifications for configuring Rally time reminders (#14)

* Added help description for NotificationTriggers (#17)

Added help description in the event that the user's browser does not support Notification Triggers. The description contains a link to the new help page.

* Added Models based on tasks in project. (#35)

* Migrated Excel service to ExcelUtility and started workbook (#38)
tbm0115 added a commit that referenced this pull request Dec 15, 2021
* Implemented scheduled notifications for configuring Rally time reminders (#14)

* Added help description for NotificationTriggers (#17)

Added help description in the event that the user's browser does not support Notification Triggers. The description contains a link to the new help page.

* Added Models based on tasks in project. (#35)

* Migrated Excel service to ExcelUtility and started workbook (#38)

* Fixed card sizes for mobile. (#41)
tbm0115 added a commit that referenced this pull request Dec 15, 2021
* Implemented scheduled notifications for configuring Rally time reminders (#14)

* Added help description for NotificationTriggers (#17)

Added help description in the event that the user's browser does not support Notification Triggers. The description contains a link to the new help page.

* Added Models based on tasks in project. (#35)

* Migrated Excel service to ExcelUtility and started workbook (#38)

* Fixed card sizes for mobile. (#41)

* Added more text content and links to GitHub Discussions for contribution (#43)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request lotr-risetowar Related to the LOTR: Rise to War subsite subsite Flags as related to a subsite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant