openproject

Commit Graph

Author	SHA1	Message	Date
Oliver Günther	15394b615a	[40747] Set default scope for notifications to visible https://community.openproject.org/wp/40747	3 years ago
ulferts	fe06d885db	paginate relations collection The response to /api/v3/relations is paginated. The response to /api/v3/work_packages/:id/relations is not yet paginated as: * The front end currently does not handle pagination * The expected amount of relations returned is lower as it is limited to the relations from/to the one work package.	3 years ago
ulferts	5b15874827	fix pagination on projects api	3 years ago
ulferts	816057e65e	allow suppressing notifications on member create/update	3 years ago
Oliver Günther	04964f21e2	[40248] Remember the back_url when initiating auth_source_sso When the user creates a new session through auth source sso, they are being logged in through `logged_user=` which calls the login service and redirects back afterwards. But there is no back_url set, resulting in a my page redirect. https://community.openproject.org/wp/40248	3 years ago
ulferts	ebc25fd7cc	check visibility on notifications For now, the implementation is work package specific	3 years ago
ulferts	9647412702	exclude wp cfs not deemed to be filters from being created as filters	3 years ago
Oliver Günther	4a5a9050b6	[39561] Improve translation of whitelist error This makes the error more user friendly https://community.openproject.org/wp/39561	3 years ago
Markus Kahl	96e7f858c9	rubocop	3 years ago
Markus Kahl	320d1e77de	fix direct upload for all content types	3 years ago
Oliver Günther	02d503f951	Allow passing in the model name to not found for easier matching In case we don't have an exception, but want to show 404 anyway due to permissions, there is no exception object to use. Allowing to pass in the model in scope helps with the message output	3 years ago
Oliver Günther	94e2d81201	Remove unused properties of not found example	3 years ago
Oliver Günther	9a3d822b35	[38842] Allow customized not found message https://community.openproject.org/wp/38842	3 years ago
Oliver Günther	1c9f677029	Fix spec	3 years ago
ulferts	92cc5a603e	remove method that used to convert to tzinfo	3 years ago
ulferts	bbf05016b4	prevent non work package notifications to leak via the API	3 years ago
Oliver Günther	b91faeac6a	Rename remainders of reason_ian to reason	3 years ago
Oliver Günther	e3913e61e4	Remove read_mail, reason_mail, reason_mail_digest	3 years ago
Oliver Günther	11063f7a4f	If CORS is enabled, also output headers for OAauth endpoints	3 years ago
ulferts	b3b3b80cb9	eager loading for notifications and activities Associations that cannot be loaded via rails` eager loading are eager loaded by hand: * The 'data' association of journals * The 'predecessor' association of journals * The 'journable' association of journals * The 'resource' association of notifiations Since the notifications and the activities index api have a similar data structure, notifications can reuse the eager loading of activities. Does not include any optimization for the rendering of the details as of now since the structure of the details is still subject to change.	3 years ago
ulferts	80718153d7	adding reason filter to notifications api	3 years ago
ulferts	96c792c0c3	adding project filter to notifications api	3 years ago
ulferts	f0fb2fbbed	split up notification reason involved Differentiate between the receiver being assigned or accountable.	3 years ago
Oliver Günther	0ed37a6e8a	Fix specs	3 years ago
Oliver Günther	f60b86da2d	Add parser error if invalid metadata json	3 years ago
Oliver Günther	b94b0e630d	Fix filesize validation on prepared uploads	3 years ago
Oliver Günther	087360bad0	We no longer raise a multipart error when metadata is empty	3 years ago
Oliver Günther	2a8bb15d80	Fix expected validation message	3 years ago
ulferts	32d2602d76	remove individual work package mails (added/updated)	3 years ago
Oliver Günther	e5dec2dc31	Add notification group_by for projects with transformed keys	3 years ago
Oliver Günther	e9ac166593	Add notification api specs for grouping	3 years ago
ulferts	48292af26d	limit reminder times to full hours The UI already limits the times a daily reminder can be configured to be delivered to be a full hour but a different client could specify a different value which the job delivering the reminders will not support	3 years ago
ulferts	57cfa51d19	accept HH:MM as valid times for the reminders	3 years ago
Oliver Günther	ccfa29c728	Move license and copyright docs to root, fix names and references	3 years ago
ulferts	1f024e5796	remove no_self_notified user preference	3 years ago
ulferts	63ad2e8dd1	Generate IAN near real time (#9570 ) * destroy notifications upon journal destruction * breakup spec by journable * have a dedicated job for scheduling journal completed events * generate notifications in delayed job * route wiki journal events through notifications * combine mail sending for notifications into single job * robustness if a journal is deleted * linting * differentiate BaseJournal and AssociatedJournal BaseJournal holds the data of a journal that reflects the columns in the journaled model while AssociatedJournal reflects the columns of a model associated to the journaled model via a has_many/one association * route news journal events through notifications This does not yet include notifications for comments on news * include news comment in notification workflow * have the whole workflow in a single state machine job * include forum message in notification workflow * include document in notification workflow * less dependencies from acts as watchable * fix, test and document workflow job	3 years ago
ulferts	8dfba679a6	introduce resourceId and resourceType filter These filters in combination allow filtering notifications via the associated resource, e.g. a work package. When only using the id filter, due to the polymorphic association of the resource, it will not be guaranteed to only return the desired notifications. A different resource, e.g. a meeting might have the same id. But using the resourceType filter by itself also makes sense so that a client can fetch only notifications associated to a subset of resources, e.g. work packages.	3 years ago
ulferts	34467120c3	introduce resourceId and resourceType filter These filters in combination allow filtering notifications via the associated resource, e.g. a work package. When only using the id filter, due to the polymorphic association of the resource, it will not be guaranteed to only return the desired notifications. A different resource, e.g. a meeting might have the same id. But using the resourceType filter by itself also makes sense so that a client can fetch only notifications associated to a subset of resources, e.g. work packages.	3 years ago
Oliver Günther	b415c3771f	Fix specs using a nil user context	3 years ago
Oliver Günther	31726649d4	[38062] Avoid sending notifications with no ian reason	3 years ago
Oliver Günther	351cc7f969	[38062] Avoid sending notifications with no ian reason	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	e9d7c374cc	[37868] Whitelist for attachment mime types and extensions on upload (#9431 ) * Add setting for whitelist * Make attachments API BaseServices compatible * Add prepare service and contract * Correctly pass the filename to the UploadedFile * Add presence check to filename * Fix expected validation message * We no longer raise a multipart error when metadata is empty * Fix filesize validation on prepared uploads * Add parser error if invalid metadata json * When attachment is not saved, use filename property * Return correct error message on JSON parser erroro * Fix specs * Use attachment upload representer * Fix direct uploads mocks with new service layer * Lint * Fix export job using attachment service * Fix IFC controller using attachment prepare service * Fix export job * RenameRename params_getter to params_source * Fix mail handler using attachment service * Fix usage of attachment create service in documents * Reuse shared examples for document attachment spec * Fix stubbed attachment service in export job spec * Use admin user in backup spec * Fix export job for bim * Fix attachment integration spec * Fix issues_controller spec * Make budget resource spec reuse common examples * Fix attachment parsing representer spec * Replace prepare part of attachment spec into separate service spec * Clear cache for login spec * Convert document create/update into services * Budget services * Allow options to be passed to property twin * Remove setting author on budget initialize * Replace meetings update with services * Replace ifc models attachment handling with services * Don't check uploader if changed by system * Fix uploader being changed by system * Replace wiki page attach_files with attachable services * Replace avatar saving * Replace snapshot attach_files * Skip double validation when container present * Set snapshot through attachment service * Remove attach_files * Validate content type in contract * Enforce writing the content type without accepting user input * Expect changed content_type * Fix content of viewpoint image to get correct content type * Fix tsv spec * Add create contract spec * Bypass whitelist in internal services when conflicting with user * Fix expects in specs after whitelist bypass * Render contract errors for wiki * Add before_hook to bodied to allow to pre-authorize permissions * Budget errors from contract * Document errors from contract	3 years ago
ulferts	0666c89828	Feature/35708 wp typeahead filter (#9426 ) * breakup wp resource specs into separate files * introduce and employ typeahead filter on work packages Co-authored-by: Oliver Günther <mail@oliverguenther.de>	3 years ago
ulferts	f8b28b300c	treat nil value as non false The default BooleanEquals filter treats null same as false. Because of that, notifications that are not meant for ian were displayed. The BooleanEqualsStrict filter only filters for false as expected so that the notifications are not displayed.	3 years ago
ulferts	261b0fa5c3	add bulk read/unread enpoint for notifications	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	2f00810e92	Bug/35510 editing project settings removes link to parent project for certain permission constellations (#9336 ) * breakup spec file into individual files * spec changing project without changing parent * have spec specific to project representer rendering * prevent parent from being embedded if user is not allowed to see it * extract urn into constant * handle parsing of a hidden uri * commit hidden uri * extract methods * improve understandability of undisclosed parent property * extract method to increase understandability * centralize information on default/none option * Remove unnecessary cast and cleaner structure * adapt expected fieldset in spec Co-authored-by: Oliver Günther <mail@oliverguenther.de>	3 years ago
Oliver Günther	8ace160470	[37510] Fix accesing nil fragment in custom field injector (#9335 ) https://community.openproject.org/wp/37510 This got broken in `065dfbddc4` when changing the nil parsing of representable	3 years ago

1 2 3 4 5 ...

749 Commits (c715443704ba2e7c5ebc6b19b4ef0bec5f202185)