83 Commits (61357f50918fb20e841b04788f96a2b49d0e8df0)

Author	SHA1	Message	Date
Oliver Günther	b7874852ec	Delete the notification when the WP gets destroyed	3 years ago
Oliver Günther	bcdc67cd52	[38078] Fix bulk controller not setting send_notifications ... With the bulk controller not setting notifications, the default is send_notification = true. All subsequent calls to disable notification sending is ignored, so the call in the WorkPackages::UpdateService is ignored. This PR also renames the "Send emails" option to the more appropriate term now. https://community.openproject.org/work_packages/38078	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	0c2d87b4eb	choose wp type by position only (#9297) ... There was a misconception in the code related to the `is_default` field. The `is_default` field only governs whether a type is enabled by default in a new project or not.	4 years ago
Oliver Günther	f50297227b	[34444] Projects copy APIv3 (#9149) ... * Create copy project endpoint * Add representers * Add _meta representer for copy module information * Add Meta payload representer * Extract parsing of copy attributes into service * Extract enqueue job for projects copy * Keep request object available in the bodied endpoint this allows us to access grape (e.g., for redirecting) * Add DelayedModify endpoint that redirects to job status * Use DelayedModify endpoint for copying projects * Add api paths for form/copy * Disable cache on ProjectCopyPayload * Add spec * Add resource spec * Extend schema for meta * Extend docs * Don't pass service result, but state as meta to forms * Use copy dependencies for naming copy options in API * Add description property for counting * Remove unused action attribute * Skip writable checks on meta property for payloads * Use prepend to allow create(...) usage with meta * Extend copy spec with custom fields usage * Add spec for copy flags * Add todos * Pass errors correctly to copy settings * Remove invalid parameters to copy that are now caught through project service * Remove duplicated validation * Linting * Make count a human readable, formattable description string * Make source count string readable * Provide the source project for generating the counts * Extract copying of attachments into sepearate dependent services This will allow the API to dynamically generate a copy association schema from each dependent service * Add spec for copy schema representer * Add payload representer spec * Also validate the model to pass the validations when copying * Make description optional and add note to schemas.apib * Add send_notifications to representer * Allow to configure sendNotifications * Allow bodied to receive a state process callback That allows us to manage the meta state on the API level, not on the service level * Make other services compatible with BaseCallable * Fix params passing from BaseCallable Wrapping a single hash into kwargs obviously loses their key indifference and we can't expect all services to use a single params object. Older services use kwargs * Copy wiki page with parent_id This will save some memory instead of memoizing the entire wiki page * Fix typo * Default to true for all copy associations That means if only a true value is passed, all other options will still be true * Do not try to copy attachments if the base dependency wasnt copied * Ensure null identifier gets rendered * Fix paths to the form and commit * Change redirect to an URL * Extend documentation on meta properties as table * Ensure we pass a default params if incoming params are nil Now that SetAttributes needs to splat params, they do no longer correctly get the no-param-default behavior of Ruby, so we need to explictly assign a default params hash * Fix expect for trailing path	4 years ago
Oliver Günther	673d8528c3	Reload work package before destruction (#9116) ... https://sentry.openproject.com/sentry/saas-openproject-prod/issues/12403	4 years ago
ulferts	7ecf08e005	rework changed_by_system (#9043) ... Before, it was implemented by passing the changed attribut keys over to the contract to whitelist them. This lead to: * The contract interface becoming bloated * Having to rely on the knowledge of the developer not to falsely whitelist an attribute. The developer would also have to make sure to not perform a mass assignment after the attribute has been whitelisted The new approach it to integrate the behaviour into the model which is first altered in the service before it is scrutinized in the contract. The information about the changed attributes is now stored inside the model which removes the necessity to flag the whitelisted attribute separately. Additionally, the exact change is tracked. So if an attribute is set to one value inside a whitelisted block there is no risk in later on performing a mass assignment. This comes at the cost of extending the models which is weird also it is build into the default SetAttributesService so child classes do not have to worry. One might include the module into every AR model but currently we only need it for a very specific use case.	4 years ago
ulferts	1bdd2ab9ae	safe automatic fixes by rubocop (#8994)	4 years ago
luz paz	3683042c9b	Fix typos in user facing and non-user facing code ... Found via `codespell v2.1.dev0`	4 years ago
ulferts	6140f4c7e9	update copyright to 2021 (#8925) ... Updates the copyright to 2021 for all files that have a copyright. Files in our source code without the copyright header still do not receive one automatically. Additionally, backlisted files are also excluded. Previously the copyright of chiliproject which references redmine stated a copyright of redmine up to and including 2017 which is not true for the code we have in here. Because of that I changed that to 2013	4 years ago
ulferts	bf6d0c727a	fix scheduling bug if predecessor dates are removed and follower only has due date set	4 years ago
ulferts	a112999dcc	fix ruby 2.7 deprecations (#8867)	4 years ago
ulferts	f511f8d4f5	set default role when copying project as non admin (#8824)	4 years ago
ulferts	d77fa1aeb6	fix date inheritance from parents with only one date available	4 years ago
ulferts	b8bb7059a1	avoid stale object errors on bulk edits	4 years ago
ulferts	05c9e3fca6	Feature/default dates for children (#8605) ... * fix start_date expectation based on setting * extract methods * set dates of parent work package * pass parent on form request * have initial dates displayed correctly in wp create form	4 years ago
ulferts	74fe191997	handle switching from manual to automatic	4 years ago
ulferts	7065c162fd	disregard manually scheduled nodes when scheduling work package graphs ... Instead of fetching all work packages from a subset of the whole graph constructed by hierarchy and follows relationships upon scheduling, only work packages eligible for automatic rescheduling are returned. In order to avoid quering the database multiple times, the whole subgraph is returned at once using a recursive CTE.	4 years ago
ulferts	cd85434ea6	aaj no longer requires loading a lot of associated objects	4 years ago
Oliver Günther	01954c26bc	Refactor copy services to use shared state	4 years ago
ulferts	9c9ef5b6fd	method extraction	4 years ago
ulferts	f612752b9a	handle erroneously missing start_date when scheduling	4 years ago
Oliver Günther	373b673f1a	Replace export modal with job status modal	4 years ago
Oliver Günther	0ca1865dd9	Add reference to user in status, extend enum, and add payload ... - Adds the user reference directly to a job status, this avoids having a separate object for many cases - Adds a payload object to the status to transport information - Extends the enum with a type when the job was cancelled	4 years ago
ulferts	e20910a2c9	remove unused service ... The last call to the `reschedule_after` method was removed in #690e0ffc60.	5 years ago
ulferts	3ccfb86dfe	correctly set initial notification value on work package copying	5 years ago
ulferts	67ade5002d	extract journal notification setting to separate class	5 years ago
Oliver Günther	f10e5bcafc	[33111] Always assign query_props to change persisted queries ... In case of a persisted query, the export job only received that query without any (unsaved) query_props, resulting in changes to it never being applied. Since a saved query is only passed for manual sorting, the problem only occurs there. https://community.openproject.com/wp/33111	5 years ago
ulferts	4cb0d0ff91	Feature/delayed bcf export (#8275) ... * delayed bcf export * remove unnecessary eager loading * Update rxjs polling statement with loading indicator * Avoid error handling due to JSON parsing * delay all wp list exports Co-authored-by: Oliver Günther <mail@oliverguenther.de> [ci skip]	5 years ago
ulferts	0b15db3c9a	rename fixed_version to version	5 years ago
ulferts	2bc34e5006	deprecate attach_files	5 years ago
ulferts	5ad5905762	adapt in_context calls	5 years ago
ulferts	0cda1869a8	employ advisory lock for all update services	5 years ago
ulferts	399e8c3ed9	rename according to zeitwerk	5 years ago
ulferts	9e2287bfcc	rename and inflect according to zeitwerk	5 years ago
ulferts	24c26702bb	factor in non leaf descendants when calculating the derived hours ... Before, only the leaves where taken into account to calcluate the derived estimated hours. With the hierarchy of Parent Child Grandchild Only times from "Grandchild" would be propagated to "Parent" with the ones on "Child" being ignored. We now factor in all descendants	5 years ago
ulferts	13557f13ae	increase version check resilience	5 years ago
ulferts	fa7e469051	also update attributes when parent_id is signaled to be updated	5 years ago
Henriette Dinger	bd7f4e4814	Update copyright notice	5 years ago
ulferts	e594f76cc6	have authorization in topics representer	5 years ago
ulferts	6c4e572f4e	consolidate allowed_* logic in the wp contract	5 years ago
ulferts	bc682f6b01	bail on inexistent related resource ... * status * type * assignee * priority	5 years ago
ulferts	dccd6c0b92	wip - bcf api topic#create	5 years ago
ulferts	4eab6effe1	merge changed_by_system into contract options	5 years ago
ulferts	05ac3af1ee	add projects delete end point to api	5 years ago
ulferts	57f7da4f39	reuse project services in controller	5 years ago
ulferts	f46c58faf9	use the wp payload instead of the resource ... That way, local changes will not be ported over between states except when the user has explicitly made them. E.g. changes by the user to the subject should be ported over, but changes to the description, caused by a default description, should not	5 years ago
Oliver Günther	edb1fbba79	[30816] Allow moving follower with unset due date ... https://community.openproject.com/wp/30816	5 years ago
ulferts	c3c197bc24	attachments for custom_text widgets on dashboard	5 years ago
Markus Kahl	cb6d28136d	allow editing estimated time for parents, show derived separately	5 years ago