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

feat: link to add events from agenda to google calendar #62

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rabarbra
Copy link
Contributor

@rabarbra rabarbra commented Aug 16, 2023

Screen Shot 2023-08-16 at 9 23 27 PM Screen Shot 2023-08-16 at 9 25 05 PM

@rabarbra
Copy link
Contributor Author

I need to add an item to setting to enable / disable this feature.

@FreekBes
Copy link
Owner

FreekBes commented Aug 16, 2023

For adding a setting, please take a look at the following commits / releases:
https://github.com/FreekBes/improved_intra_server/releases/tag/v3.2.0 - adding a setting to the database
FreekBes/improved_intra_server@fe0453c - adding a setting to the back-end (setting up a back-end dev environment is rather difficult due to not having set up a Dockerfile yet - it's on my todo list)
a7f746f - adding a default value to the extension
58ffd8e - retrieving the value set from the storage

I'll try to take a look at and test your other pull requests soon. Thanks a lot for your contributions.

Speaking of contributions, through Improved Intra it's possible to get a "Contributor" title added to your Intra profile, though it is only visisble for Improved Intra users. If you would like to have this title, please let me know. Sadly it cannot be equipped like any other Intra title - it is hardcoded into the extension.

@FreekBes FreekBes added the feature request New feature or request label Aug 21, 2023
@FreekBes
Copy link
Owner

I've set up a branch that runs the Improved Intra back-end server in a Docker container. This should make development and testing of the to-be-added setting easier.

However, I could also implement it myself. Let me know if you want me to.

@rabarbra
Copy link
Contributor Author

Thank you very much for the comments and for the backend  branch. I don't mind doing this backend part myself, but a little bit later - now I have to work on another project for a while.

As for the suggestion to add a "Contributor" title to my intra profile - I don't mind, it would be great. My intra name is psimonen.

FreekBes added a commit that referenced this pull request Nov 5, 2023
and fix :not() selector for various logins
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants