58865 Commits (2fae8e432b4afb93aa23ce29d2e23623f53db654)

 

All Branches   

Search

Author	SHA1	Message	Date
Oliver Günther	2fae8e432b	Remove inherit align for avatar ... [ci skip]	4 years ago
Oliver Günther	3429161e14	Allow to use puma for packager again (#9180)	4 years ago
Oliver Günther	d98f08b7db	Use tagged logger with timestamp formatting as early as possible (#9178) ... This ensure identical behavior in all environments and results in loggers receiving timestamps and levels in development as well.	4 years ago
Oliver Günther	395cf2a5d1	Remove proto plugin accidentally committed	4 years ago
Oliver Günther	6dc662b1e4	[36243][30226] Different avatars for placeholders and groups (#9169) ... * Fix members autocompleter to allow hrefs * Add rendering of different avatar types * Remove unused code in backend avatar rendering * Add very basic test that the avatar gets customized per type * Allow avatar rendering in the CreateAutocompleter * Restructure avatars into BEM css * Remove avatar-classes in favor of passing size from backend * Remove to avoid confusion We use tracking with both href and , and was introduced to avoid a property called href to be overridden, but that was never a problem before. So from now we can safely use .href * Fix broken principal avatar in user activity * Ensure filters get updated even if the form is not reloaded There was a bug in conditionallyLoadForm that checked the property of link (always undefined) and would thus always reload a form. What we actually need is a form that gets reloaded only when the query link changes, but still update services from the updated query, such as filters * Adapt specs to changes classes * Rename op-principal-avatar to op-avatar	4 years ago
ulferts	a25f6c5cc1	Fix/bump gems (#9175) ... * bump acts_as_list * bump aws-partitions * bump plaintext * bump rubocop	4 years ago
ulferts	21df5e46b4	Merge branch 'release/11.2' into dev	4 years ago
Benjamin Bädorf	68949daa03	Feature/35521 ium capabilities (#9158) ... * Started experimenting with capabilities * Started adding akita store * Use current user store to filter projects * Work on integrating capabilities * Added capability checks * Fix project select * Working on ium specs * Change ng select option list format * Fix proejct search label * Fix some specs * Fix issue with principal select * Add spec for missing placeholder user capabilities * Add new capability specs, fix existing ones * Fix import * Update shrinkwrap * Add JSDoc deprecations Co-authored-by: Oliver Günther <mail@oliverguenther.de>	4 years ago
OpenProject Actions CI	a723802985	update locales from crowdin [ci skip]	4 years ago
Matthew Trescott	1af65d3b6c	Replace custom i18n function in committers settings ... Missed in #8679. See also #8681	4 years ago
ulferts	aeed35fdab	sort files when requiring	4 years ago
ulferts	295fc52166	avoid setting spilling over to next spec	4 years ago
ulferts	651f115300	limit any_name_attribute filter operators ... `=` and `!` used to be supported but resulted in comparing the concatenation of first/last-name, login and mail as a whole and not individually. As the filter is mostly used as a fuzzy search, it is ok to only support contains and not contains.	4 years ago
Oliver Günther	174427fc05	Remove duplicate not_locked scope (#9166) ... * Remove duplicate not_locked scope The enum itself creates scopes but they do not include not_builtion. So instead, let's create these scopes ourselves * Change enterprise spec to no longer test for builtin users	4 years ago
ulferts	9fa5599392	Feature/member notifications (#8958) ... * spec with correctly scoped links * move db check into own file - fix deprecation * basic spec for member creation service * use constants for all notifications * send an OP notification after member has been created * send an OP notification after member has been updated * mails on group member added Depending on whether the membership existed before or not, an updated or a created notification is send. This is done asynchronously. * move all mail sender background jobs into namespace * wip * wip * correct handling group member notifications * add setting enable/disable mail sending on member alterations * use services in members controller * move Notifiable to OpenProject * remove member after save hooks * cleanup/testing/linting * render member mails in receiver locale * remove add_member! method * use mailer layout for all mailers * Update app/services/groups/cleanup_inherited_roles_service.rb Co-authored-by: Oliver Günther <mail@oliverguenther.de> * use around callback to avoid prepending * handle nil params Co-authored-by: Oliver Günther <mail@oliverguenther.de>	4 years ago
Benjamin Bädorf	3f3e831f6a	Add any name attribute filter to principal resource collection (#9170)	4 years ago
OpenProject Actions CI	ecbb81b5a6	update locales from crowdin [ci skip]	4 years ago
Benjamin Bädorf	64143001fc	Set moment locale globally (#9040) ... * Set moment locale globally This sets the moment locale to the application locale globally, ensuring that moment uses the right translation strings wherever applicable. However, since we only work with languages (en), and not real locales (en-US), the first day of year calculation has to default to some option. This commit defaults to ISO 8601 for every language and locale. * Add setting to mark dayOfYear * Fix NaN check * Add spec for switching settings * Output uniq github specs Co-authored-by: Oliver Günther <mail@oliverguenther.de>	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
ulferts	901cbada65	Feature/gem update task (#9165) ... * task for updating gems * task for updating used rubocop on codeclimate * combine into default update task * bump aws-partitions * bump aws-sdk-core * bump aws-sdk-s3 * bump bootsnap * bump debug_inspector & i18n & loofah & nokogiri * bump delayed_job_active_record * bump doorkeeper * bump dry-logic * bump excon * bump connection_pool & faraday & faraday-excon & faraday-net_http_persistent & net-http-persistent * bump fog-aws * bump marcel * bump nokogumbo * bump oj * bump parallel_tests & ruby-saml * bump parser * bump prawn-markup * bump rexml * bump rspec-rails * bump rubocop * bump sentry-delayed_job * bump sentry-rails * bump sentry-ruby & sentry-ruby-core * bump unicorn & unicorn-worker-killer * bump webmock * use rubocop-1-10-0 on codeclimate * whitelist i18n patch * bump listen * rename update task to avoid misconceptions * bump faraday & faraday-excon & faraday-net_http_persistent	4 years ago
Oliver Günther	16c1617696	Merge remote-tracking branch 'origin/release/11.2' into dev	4 years ago
Oliver Günther	412302a093	Better database connection check ... Fixes the MySQL output and adds a rake task to properly check the connection This will e.g., output something like this ``` Database connection failed with error: FATAL: Datenbank »bla« existiert nicht ```	4 years ago
OpenProject Actions CI	65dc28d913	update locales from crowdin [ci skip]	4 years ago
OpenProject Actions CI	12cf765431	update locales from crowdin [ci skip]	4 years ago
OpenProject Actions CI	8b763cb038	update locales from crowdin [ci skip]	4 years ago
ulferts	8e834072ed	adapt spec	4 years ago
ulferts	3d60d26d8c	replace legacy spec	4 years ago
ulferts	c0784d8e5f	remove covered legacy spec	4 years ago
ML-OpenP	fefe8731be	Docs for new permissions of 11.2 (#9092) ... * small corrections * changed link * changes for new global permissions * working in review comments (part 1) * working in review comments (part 2)	4 years ago
Oliver Günther	3e723a07af	[36764] Fix margins of table when sums and/or inline create showing ... In https://github.com/opf/openproject/pull/8771, a double styling was added when the inline create row is showing. Back then, sums row was always taking up space even if not visible so the double margin was correct. This however is no longer the case. We need to apply different margins when either sums or inline create, or both are showing. https://community.openproject.org/wp/36764	4 years ago
ulferts	5ad8fae4ab	add task to print all api routes (#9161)	4 years ago
Philipp Tessenow	bd906716e5	[#36153] link manually created pull request to a work package using the GitHub integration plug-in (#9070) ... * Receiving and Parsing GitHub webhooks * added services for upserting PullRequests, GitHub users, and, checks. * added the datamodels for storing these entities * added logic to parse GitHub webhooks and save the entities above * fix some specs, added model specs * let UpsertPartialPullRequest return the upserted PR * added the last missing specs * avoid double-posts when editing pull requestsx * attempt to reduce flakiness * added pull requests API endpoint * handle edge case of converting partial PRs to full PRs * name raw input of an upsert service 'payload' to differentiate it from the prepared params * handle edge case: partial upsert after full PR creation * github_user does not reference users anymore * fixed a doc comment * use enums instead of custom validations * WIP * pairing with jens and some cleanup and test writing * better migration, clear up old PRs with a cron job * handle case of non-edit events which do not introduce new linked work packages, but still require a comment	4 years ago
Wieland Lindenthal	21af36f56a	Add step for quick add menu to onboarding tour	4 years ago
OpenProject Actions CI	8c44d99d76	update locales from crowdin [ci skip]	4 years ago
Wieland Lindenthal	c29df6223e	Change icon for inviting a user from within drop down from plus to user-plus [ci skip]	4 years ago
Oliver Günther	2d22930bdb	[36746] Unset plain password accessor after saving (#9157) ... * [36746] Unset plain password accessor after saving Otherwise, this password remains and is validated again due to "user.register!" which then fails with an exception https://community.openproject.org/wp/36746 * Don't try to use user.password after saving it now gets cleared * Remove user.password usage in specs The user password attribute is an accessor that now gets deleted after saving so tests can no longer depend on it being present after created from a Factory * Also retain user password in mail handler	4 years ago
OpenProject Actions CI	a75ff8534e	update locales from crowdin [ci skip]	4 years ago
OpenProject Actions CI	4f6ea6fbf0	update locales from crowdin [ci skip]	4 years ago
Oliver Günther	2b0bce6b5c	Improve message when nulldb is being used (#9156)	4 years ago
Oliver Günther	8f5006a51b	Rename ActiveRecord class to UserSession for clarity	4 years ago
Oliver Günther	aeec219bfb	Fix connection being memoized in sqlbypass class	4 years ago
Oliver Günther	27e8c1cdb0	Ignore health checks in sentry logger trace	4 years ago
OpenProject Actions CI	b5ae51e5d0	update locales from crowdin [ci skip]	4 years ago
Oliver Günther	daa0cfdee9	Ensure we're not memoizing the connection (#9153)	4 years ago
Philipp Tessenow	fd19962429	Fixing a flaky specs in the github integration feature specs (#9151) ... Sorry for introducing it, it was a false assumption about how work package subjects are generated in the factory. Turns out it is more stable and a better test to specify the subject we test in the test setup itself anyways.	4 years ago
Oliver Günther	16446cd80b	Avoid logging with log delegator in migrations (#9150) ... If the slow log catches during migrations, accessing something like User.current will fail as not all tables are defined yet.	4 years ago
ulferts	aa4397cf28	Merge branch 'release/11.2' into dev	4 years ago
Oliver Günther	5744dcd422	Explicitly require and use Sentry::SendEventJob	4 years ago
OpenProject Actions CI	ef760c9cd0	update locales from crowdin [ci skip]	4 years ago
OpenProject Actions CI	42db620da8	update locales from crowdin [ci skip]	4 years ago