1008 Commits (328c627fa76294250fcd655b2f7ecbae3bcd66a6)

Author	SHA1	Message	Date
Oliver Günther	a776dff0b4	Add index on enabled boolean	3 years ago
Eric Schubert	c8a35f97da	[39451] remove base64 snapshot data from bcf vp ... - https://community.openproject.org/work_packages/39451 - remove base64 data after converted to snapshot attachement and before storing into database - added migration to remove snapshot data from current data	3 years ago
Markus Kahl	379a0969bb	handle invalid group_users entries, prevent new ones	3 years ago
ulferts	a33fcbac94	user_default_timezone setting is first fallback for reminder mail	3 years ago
ulferts	4b155634eb	migrate existing time zones to tzinfo format	3 years ago
Oliver Günther	19844f2b96	Linting	3 years ago
Oliver Günther	385840ec89	Implemennt workdays component	3 years ago
Oliver Günther	83ad86ee7d	Add workdays index and default values	3 years ago
Oliver Günther	528ca50faa	Strip sent_mail into two columns mail_alert_sent and mail_reminder_sent	3 years ago
Oliver Günther	8dec991d01	Remove channel from non-wp PR	3 years ago
Oliver Günther	5b8df97498	Rename read_mail_digest into sent_mail and merge with non-wp items	3 years ago
Oliver Günther	ecfa4a765f	Remove NotificationSetting#channel and all	3 years ago
Oliver Günther	e3913e61e4	Remove read_mail, reason_mail, reason_mail_digest	3 years ago
ulferts	0c947da528	remove invalid notifications before setting constraint	3 years ago
Oliver Günther	903e899c3b	Remove notified_events usage in settings	3 years ago
Oliver Günther	e677bfb4c2	Add new notification settings for non-wp settings	3 years ago
ulferts	98d74e00de	ensure notifications are linked to existing journals ... There can be a race condition when a user continues to alter a work package or other resource after a first change has been made resulting in a journal and the background process creating a notification for that journal. If the second journal caused by the users continued interaction removes the first journal after the job for creating a notification has already started, all checks within ruby are blind. Therefore a foreign key constraint is needed.	3 years ago
Wieland Lindenthal	178e8abe50	WIP Add migration for flipping clipping planes that were created in OP (#9682)	3 years ago
Eric Schubert	d6b87a2210	[#38833] BCF API Comments Service ... - https://community.openproject.org/work_packages/38833 - added requests tests - added check for expected body length - added representer and api for bcf_comments - added create and set attributes service for bcf_comments - added migration for comment hierarchy - added create and update service for journals	3 years ago
ulferts	57ed3c8fd3	add indices to support finding notification recipients	3 years ago
Oliver Günther	6647c4489f	[38712] Immediate reminder options	3 years ago
Oliver Günther	614de339f0	Remove all unused keys from the settings json	3 years ago
ulferts	062e33be30	have boolean values for warn_on_leaving_unsaved	3 years ago
ulferts	0cee545248	adapt the json_validator from activerecord_json_validator ... The error messages need to be more fine grained than what can be achieved with the original gem	3 years ago
Oliver Günther	ccfa29c728	Move license and copyright docs to root, fix names and references	3 years ago
ulferts	d7c399320b	rename batch size env var for migrations	3 years ago
Oliver Günther	1fa9230c8e	Fix default value for hide_mail	3 years ago
Oliver Günther	a75bbb078a	Migrate UserPreference to JSON storage	3 years ago
ulferts	2a6ab15101	add work package scheduled notification setting	3 years ago
ulferts	aabe286e4b	add index on notification settings columns expected to have be distributed unevenly	3 years ago
ulferts	8cfb94be7d	remove work_package_updated from notified_events	3 years ago
ulferts	09f1a37f3c	add work package prioritized notification setting	3 years ago
ulferts	61357f5091	add work package processed notification setting	3 years ago
ulferts	a1074bcb93	add work package created notification setting	3 years ago
ulferts	f246346014	add work package commented notification setting	3 years ago
Oliver Günther	db401c553f	Fix executing the removal of journal_versions ... The original version does not work correctly.	3 years ago
ulferts	13e418e34e	env var for aggregate journal migration batch size	3 years ago
ulferts	b88fe2349a	increase author as watchers migration performance ... Instead of instantiating every work package and relying on the watcher model to check whether the adding the user as a watcher is allowed, the code now uses plain SQL	3 years ago
ulferts	e506993c6b	improve performance of polymporphic journal data migration ... Removing the indices before inserting the data and then readding them later on is faster than updating them while inserting	3 years ago
ulferts	b3bd780c2c	improve performance of aggregate journals migration ... The purpose of the rather low batch size (the default is 1000) does no longer apply. It was important before the journals where cast to an array in the next line.	3 years ago
Wieland Lindenthal	496253d029	WIP Add migration for flipping clipping planes that were created in OP (#9446)	3 years ago
ulferts	75bb809ad6	Feature/1899 send daily email summaries (#9430) ... * email digests * use time for notification digest setting * safeguard against empty digest	3 years ago
Oliver Günther	991380b7e5	Extend error message when migration fails due to two schemas	3 years ago
Oliver Günther	a8652d7483	Remove length restriction on CustomField#name	3 years ago
Oliver Günther	9636a08988	[26688] In-app notifications (#9399) ... * Add bell icon to icon font * Add in app notification in top menu * Add fullscreen modal * Add notification modal and items * Style items * Toggle details of item * Mark all read * Add no results box * wip specification for event api * Add events table, query and index * Send out events from WP notification mailer job There we have the recipients present * Add cleanup job for older events with a setting * Hide bell notification when not logged * Add specs for events API index/show * Fix setting yml key * remove pry in event creation * Fix before hook in events API to after_validation * Fix polymorphic association raising exception for aggregated journals * Fix typo in read_ian * Fix yml entry for mentioned * Add read/unread post actions to event API and add specs * Wire up API to frontend * Fix order on events * Switch to unread in notification * Add event query * rename WPEventService * route wp mail sending over events * rename spec methods * author becomes watcher * correct message call signature * rename events to notifications * renname parameter to reflect notification nature * create author watcher for existing work packages * Merge unreadCount from store * Take a stab at polymorphic representers * Fix link generation in polymorphic resources For journals, no title is being generated however * Fix frontend model for context * Use timer for polling * add notification_setting data layer * Fix show resource spec * Fix duplicate class in notification bell item * Add minimal feature spec for notification * API for notification settings * Persist notifications * adapt work package notification creation to notification settings * extract notified_on_all * consolidate wp#recipients * concentrate wp notification in journal service * simplify methods * Remove unused patch endpoint * Add specs for rendering and parsing notification settings * Contract spec * Update service spec * adapt specs * Angular notifications frontend commit e29dced646 Author: Wieland Lindenthal <w.lindenthal@forkmerge.com> Date: Mon Jun 21 17:34:50 2021 +0200 Create Akita store and query for notification settings commit 1a45c26c1a Author: Wieland Lindenthal <w.lindenthal@forkmerge.com> Date: Mon Jun 21 11:09:25 2021 +0200 Remove tabs from notificaition settings page commit 0ea21e90c1 Author: Oliver Günther <mail@oliverguenther.de> Date: Sun Jun 20 21:55:48 2021 +0200 WIP in app settings * migrate notification data * add project visible filter to project query * Add inline-create and table display grouped by project * Add notifications under admin/users * Remove notifications partial * Rename notififcations store to user preferences store * Add setting for self_notified and hook that up to the backend * Add aria-label to table checkboxes * Restyle table and toolbar * replace remains of mail_notifications attribute * initialize notification settings for new user * adapt my_preferences references * reenable no self notified for documents * adapt specs * Avoid has_many :notifcation_settings Rails magically autosaves the user's preferences when the user gets saved, which somehow also tries to save the notfifications even when unchanged. This breaks some specs such as the avatar upload spec. As we can't update the assocation through rails anyway, just delegate to the user for reading instead. * Restore update method of notification settings * Restore update spec * fix spec syntax * lint scss * linting * Fix content_tag for bell icon * Add feature specs for notification settings * Disable ContentTag cop * use visible filter to get projects for notification The visible filter will reduce the project list down to the set of projects visible to the user provided as a parameter. This includes public projects. * test for actual mail sending * adapt me resource path this.apiV3Service.users.me changed its type in 0d6c0b6bc7 * Implement changed migration * Linting * Add actor to notification representer * Fix factory creating a duplicate WP journal * Add work packages loading and journal details to notification entry component * IAN basic facets, keep and expanded states. * Fix notification bell spec * Render body separately and add auto updating relative time * Add fixedTime title * Add actor to notification entry * Fix clicking links on work package and project * Tiny styling changes on entry row * Disable count in notification if larger than 99 (wont fit) * Introduce virtual scrolling to entry table * allow delaying & prevent mail sending if ain read Introduces a setting to delay mail sending after a journal aggregation time has expired. That way, users can confirm a notification in app. If they do before the delay expires, no mail is sent out additionally for that user. * consolidate notifications (in&out) into shared admin menu Co-authored-by: ulferts <jens.ulferts@googlemail.com> Co-authored-by: Wieland Lindenthal <w.lindenthal@forkmerge.com>	3 years ago
ulferts	2a015eeac3	Feature/journal aggregation on write (#9396) ... * aggregate journals on write - simple implementation * remove references to aggregated journals on read * rewrite existing journals to be aggregated * remove aggregated journal reference * only delete predecessor if it can be aggregated * fix aggregation condition * adapt specs * remove aggregated journal functionality from production code The code is still necessary in a migration for the time being * adapt specs * use polymorphic belongs_to for journal data The association can then be employed for eager loading * eager load for activity rendering	3 years ago
ulferts	c1672f239c	remove reply from message journaling (#9293) ... The columns: * last_reply_id * replies_count are columns that exist for performance reasons only. They do not contain information about the message itself. If they are journaled, adding a message to a forum updates the last_reply_id of the parent message. The user replying will then be shown to have updated the replied to message	3 years ago
Oliver Günther	245b0ebbd7	[35999] Add DB constraint on project identifiers (#9265) ... https://community.openproject.org/wp/35999	4 years ago
Oliver Günther	6c75e16dc4	[37140] Use acts_as_url to generate unique slug for identifier (#9250) ... * Use acts_as_url to generate unique slug for identifier * Re-enable project identifier spec now handled in backend * Add integration spec for set attribute service * Adapt spec for new empty identifier * Avoid double error messages for empty identifier acts_as_url may return an empty string depending on the name, but the empty presence validation will suffice in this case, we don't need the invalid validation * Fix stubbed errors object accessed by acts_as_url * Avoid trying to get the upstream unique attribute in case of errors * Remove sequential_project_identifiers setting * Adapt spec to review feedback	4 years ago
Markus Kahl	8c8b8bbfa7	create backups via UI (#9136) ... * create backups via UI * Fix import of modal service * introduced backup token and addressed remaining comments * allow disabling permissions * improvements - only make user wait to use backup token in if really necessary - notify admins of new backup token - disable 'include attachments' option in UI if unavailable - documentation - misc * spec fixes * fixed feature spec * allow setting capybara host in every case * removed unused style file * addressed review feedback, added further feature specs * polish (code climate) * Avoid empty attachments * Don't raise filesize validation for internal exports Co-authored-by: Oliver Günther <mail@oliverguenther.de>	4 years ago