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

[WIP] Calendars export to other devices (issue#824) #102

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

Seal98
Copy link
Contributor

@Seal98 Seal98 commented Aug 10, 2017

Creating a servlet that allows external programs to subscribe to the Sigmah calendar using URL

Creating a servlet for making a subscription to the URL
Creating share buttons with a popup window with the URL after its
pushing
 
Implemented: 
-Added showing share calendar events URL with parameters: 
event type and project id
-Added tool tips to share calendar buttons
-Export servlet is renamed to ExportCalendar
-Added fetching calendar events by the servlet for specified project id.
-Added generation by servlet iCal output with Sigmah calendar events.
 
Performed refactoring and code cleanup.
 
Performed refactoring and code cleanup according Codacy review remarks
Looltip for each button was changed. The functional for the events
button works
@osarrat
Copy link
Member

osarrat commented Aug 21, 2017

Hi @Seal98,

Great to see so much work on this issue #824 ! Looking forward to the demo tomorrow !
A quick feedback on commit presentation:

  • no need to start your commit message with "[WIP]" : the "[WIP]" marker is only for the Pull Request title globally, not for individual commits
  • important: add a space when typing "issue #824" in your commit title: typed as you did it, it won't be recognized by Mantis

See you tomorrow !

Added possibility to update(refresh) in Outlook imported by URL Sigmah
calendar with defined in ExportCalendar servler time interval.
Added name to imported calendar.
Added methods to get from Request User-agent and ip info for logging to
see who calls ExportCalendar servlet.
-Method getClientIpAddr is refactored to comply with Codacy check.
-IndexedDB fix to allow run Sigmah online is applied.
Hiding all Share-link buttons except the 'events' button
-Implemented URL parameter values encryption when URL is created.
-Implemented URL parameter values decryption when URL is processed by
export calendar servlet.
-Made the URL look like a filename ending with ".ics"
-Added to Set to export calendar events Time Zone, Publish, Organizer.
-Performed code changes according Codacy code check.
@ghost
Copy link

ghost commented Nov 28, 2017

At first sight, this can't be merged without a global clean-up (altered license headers, personal settings in pom.xml, offline mode disabled...).

@osarrat
Copy link
Member

osarrat commented Nov 29, 2017

Thanks @numero-six for this review !
@Seal98 , do you feel you can do those necessary changes in order to merge your pull request ? Could you by the way also merge your two pull requests which both impact the calendar ?

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