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

Opening Waiver from table doesn't fill up date #244

Closed
tupaschoal opened this issue May 25, 2020 · 1 comment · Fixed by #262
Closed

Opening Waiver from table doesn't fill up date #244

tupaschoal opened this issue May 25, 2020 · 1 comment · Fixed by #262
Labels
bug Something isn't working

Comments

@tupaschoal
Copy link
Collaborator

Describe the issue
When you click on a day to open the waiver window, the dates are not being filled up

To Reproduce

  1. Open TTL
  2. Click on any day to open the workday waiver manager
  3. Note how date is unfilled

Expected behavior
Date should be pre-filled with the clicked day

Environment

  • Version: 1.5.2-dev
  • Electron: 9.0.0
  • Chrome: 83.0.4103.64
  • Node.js: 12.14.1
  • OS: Windows_NT x64 10.0.18362

Screenshots
16ad7d82-41af-4da3-b236-12c727032301

Additional context
Opening it from the menu fills it with the current day

@tupaschoal tupaschoal added the bug Something isn't working label May 25, 2020
@NetizenAbel
Copy link
Collaborator

Let's check the difference between the call in calendar.js vs the call in main.js's menu. Ideally that entire part should be put into its own js, and reference by the same methods.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants