-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
django-sentry depends on django-paging (and on django-templatetag-sugar) #2
Comments
You commiting skills outperforms my typing skill :) |
Missed paging in INSTALLED_APPS, but great catch :) |
ckj
added a commit
that referenced
this issue
May 10, 2018
lynnagara
added a commit
that referenced
this issue
Apr 19, 2019
matejminar
added a commit
that referenced
this issue
Jan 13, 2020
* ref(ts): Convert <SidebarOrgSummary> to typescript * comment * fix: Use django's is_safe_url function (#16337) The implementation of this function from django 1.9.13 covers more invalid URLs than the one from django-sudo does. We should not redirect to `http:123347` style URLs. * fix(ui) Fix gravatar fallbacks (#16336) When updating Gravatar to typescript, I removed some property spreads as they triggered typescript warnings and the propTypes didn't cover any props that needed spreading. The onLoad and onError props were load bearing as they are used to handle gravatar fallbacks. * Removing unused class (#16278) * fix(discover) - Show errors if we encounter them loading discover - Shows a permission error if a user without access to events like a billing account tries to access discover * feat(discover) Switch trace and span id to use columns instead of context Have the trace_id and span_id use the built in columns instead of the context array. * fix(ui): Fix <ContextPickerModal> and refactor to ts (#16090) This removes `LatestContext` dependency and fixes `ContextPickerModal` and refactors it to ts. Co-Authored-By: Matej Minar <matej.minar@sentry.io> * fix(ui): Fix redirect after creating/editing Metric Alert rules (#16342) Also makes sure that the index redirects to combined alert listing view. * ref(perf) Attempt to improve performance of facets endpoint (#16359) * Fetch tag values individually. This will let snuba use the promoted columns more effectively as they perform better. * Only get 10 facets by default. * Use a having clause instead of WHERE to exclude tracing tags as it performs better. * Don't use fields that unpack into arrayJoin() expressions where possible. * Enable sampling at lower project thresholds, as we don't need full accuracy. * fix(saml2): Improve error message on attribute mapping (#16347) * ref: Remove SnubaEvent (#16309) Remove SnubaEvent since we are moving towards a single event model (eventstore/models/Event) everywhere. * fix(tasks) Add event data to user report asynchronously If an event hasn't been propagated when the user report is created, then queue a task to sync the event data once it's stored fully. * feat(django): pytest startup and test collection on Django 1.11 (#16105) * chore: cleanup bitfield (#16031) * fix(migrations): Make Snuba event migration more robust (#16364) Fixes getsentry/self-hosted#322 * feat(teams): Update open membership team permissions (#16244) * fix(migrations): Fix prerequisite for jira_ac migration (#16351) Fixes getsentry/self-hosted#332 * fix(auth): Correct path used for SSO errors (#16315) The route for sentry-organization-auth-settings was still located in the old settings path, this route is used for a redirect in the auth.helper module when there is a problem configuring SSO. This just corrects the redirect so it goes back to settings instead of 404ing. * fix: Remove beta tag on PagerDuty integration (#16344) * fix(tagstore) Don't explode on null values (#16368) The `sentry:user` tag can come back null when it is read from the promoted columns. Handling nulls allows get_tag_value_label() to be used more broadly. * ref: Remove now unused discover query code (#16306) Remove the automatic dataset detection logic from sentry. We have switched over to the snuba based implementation of this. Not having dataset selection in sentry also means we can remove some (but not all) of the supporting logic as well. * feat(ui): Require `org:write` role for Metric Alerts form (#16340) This disables all form fields when editing a Metric Alert without the `org:write` role. * ref(ts) Move BadgeDisplayName to typescript (#16361) * test: fix remaining Django 1.11 tests and make required (#16367) * fix(saml): Wrap individual idp config tabs with forms (#16373) * chore(django 1.11): bump django-crispy-forms to 1.7.2 (#16371) * translation+pluralization * removed duplicate style * removed duplicate style * fix(ui): Fixed alignment in <TogglableAddress> if it cannot be converted to relative (#16385) * ref(discover2) Test out sampling on tag values (#16387) The facets endpoint is still not fast. I want to see what kind of results can be obtained if we aggressively sample tag values. If this yields acceptable performance I want to try scaling the sampling rate based on the data volume so that we can better balance accurracy and performance. * test(acceptance): Try #2 to fix flakey onboarding test for dja… (#16345) Try #2, use a timed retry policy * fix(discover2): Fix Percy snapshots and UX issues on Querybuilder (#16291) * fix(discover2): Test fix on Percy snapshots * fix(discover2): Improve QueryBuilder for error/loading states * fix(discover2): Resize QueryBuilder when browser is resized * feat: Register option for eventstream (#16363) Register option for #16285 * feat(alerts): Remove "Related issues" from incidents (#16379) This removes the UI components that fetch and render related issues on the Incidents details view. * ref: Deprecate get_legacy_message (#16310) Move the implementation into the Pagerduty plugin since this is the only place it is used. * feat(tracing) Add spans for snuba queries (#16388) Add spans to collect more information about snuba queries. Having the referrer and the start of the query body should make tracking down slow queries in snuba easier as we have more context. * feat(workflow): Remove "Create incident" manual workflow and e… (#16375) This removes the UI and endpoint to create a manual incident. * feat: upgrade drf to 3.6.4 (#16392) * feat(alerts): Remove "Suspects" UI and endpoint (#16378) This was used on the Incidents details view, but is no longer used * ref(discover) Remove the raw field (#16390) Outputting the raw data when we also output all the formatted keys wastes space and adds visual noise. * fix(discover2): Use transaction root span only if there are no descendent spans for the ops breakdown (#16377) * feat(emotion): Upgrade emotion to v10 (#15106) * ref(ui): Refactor `cx` from emotion to `classnames` We generally use the `classnames` library to handle merging/dynamic `className`. No reason to use two different libraries. * upgrade emotion, add styled wrapper for typings (and related paths), add eslint rules for emotion, update babel plugin * Refactor usage of emotion `css` being treated as a class name string * change innerRef --> ref * refactor grid-emotion props and update to `reflexbox` * eslint for emotion-setup * refactor injectGlobal --> Global for emotion upgrade * Add <CacheProvider>, this is needed to turn speedy off for percy * refactor <HeaderItem> - fix missing props to styled components, remove innerRef and forwardRef * Change <Tooltip> logic for when to apply a wrapper, emotion causes the previously expected types to be different * random TS fixes for styled components * fix <RadioGroup> animation needing `css` * forwardRef for Switch * define a `theme.space` for rebass/grid * fix DOM attribute warnings with "loading" prop * fix settings breadcrumb - Box in `reflexbox` has different CSS than grid-emotion * change <ProviderRow> to remove `Flex` around CircleIndicator as it was squishing it on small widths * css fixes for percy * fix featureDisabled logic for custom alerts because emotion * refactor emotion9 --> emotion10 (imports) * update snapshots and fix tests * feat(drf): Add CamelSnakeSerializer (#16397) * docs(workflow): Fix link to screenshots in README.md (#16404) Fixes links to screenshots in README.md broken by 983c8c6 * ref(ts): Convert <QueryCount> to typescript (#16356) * ref(ts): Convert <SearchBar> to typescript (#16357) * ref(ts): Convert <SearchBar> to typescript * removed proptypes, placeholder optional * ref(ts): Convert getOnboardingTasks to typescript (#16348) * merge Co-authored-by: Mark Story <mark@mark-story.com> Co-authored-by: Chris Fuller <chris@iprogramstuff.com> Co-authored-by: William Mak <william@wmak.io> Co-authored-by: evanh <evanh@users.noreply.github.com> Co-authored-by: Billy Vong <billyvg@users.noreply.github.com> Co-authored-by: Evan Purkhiser <evanpurkhiser@gmail.com> Co-authored-by: Lyn Nagara <lyn.nagara@gmail.com> Co-authored-by: josh <josh@jrl.ninja> Co-authored-by: Burak Yigit Kaya <ben@byk.im> Co-authored-by: Megan Heskett <meg.heskett@gmail.com> Co-authored-by: NisanthanNanthakumar <nnutoronto2013@gmail.com> Co-authored-by: Danny Lee <leedongwei@gmail.com> Co-authored-by: Alberto Leal <mail4alberto@gmail.com> Co-authored-by: Aakash Kambuj <aakash@falkon.ai>
billyvg
added a commit
that referenced
this issue
Apr 22, 2020
This basically re-implements #13689 in a more simple way. This is possible due to other changes that we have done to issue details (#13875, #14731). This also (semi) fixes an issue with loading Issue Details with an environment in the URL. Previously, it would fetch details API serially: 1) without env and 2) with env. There would be a slight flicker between loading -> finished req #1 -> loading -> finished req #2. Now this seems to fire both at near the same time and cancels the initial request almost immediately. This is still not ideal but is an interim-fix as the ideal solution is a bit more involved, but will be on its way.
billyvg
added a commit
that referenced
this issue
Apr 23, 2020
This basically re-implements #13689 in a more simple way. This is possible due to other changes that we have done to issue details (#13875, #14731). This also (semi) fixes an issue with loading Issue Details with an environment in the URL. Previously, it would fetch details API serially: 1) without env and 2) with env. There would be a slight flicker between loading -> finished req #1 -> loading -> finished req #2. Now this seems to fire both at near the same time and cancels the initial request almost immediately. This is still not ideal but is an interim-fix as the ideal solution is a bit more involved, but will be on its way.
billyvg
added a commit
that referenced
this issue
Apr 26, 2020
This basically re-implements #13689 in a more simple way. This is possible due to other changes that we have done to issue details (#13875, #14731). This also (semi) fixes an issue with loading Issue Details with an environment in the URL. Previously, it would fetch details API serially: 1) without env and 2) with env. There would be a slight flicker between loading -> finished req #1 -> loading -> finished req #2. Now this seems to fire both at near the same time and cancels the initial request almost immediately. This is still not ideal but is an interim-fix as the ideal solution is a bit more involved, but will be on its way.
billyvg
added a commit
that referenced
this issue
Apr 27, 2020
This basically re-implements #13689 in a more simple way. This is possible due to other changes that we have done to issue details (#13875, #14731). This also (semi) fixes an issue with loading Issue Details with an environment in the URL. Previously, it would fetch details API serially: 1) without env and 2) with env. There would be a slight flicker between loading -> finished req #1 -> loading -> finished req #2. Now this seems to fire both at near the same time and cancels the initial request almost immediately. This is still not ideal but is an interim-fix as the ideal solution is a bit more involved, but will be on its way. The tests introduced in #18452 should cover the changes in this PR.
BYK
added a commit
that referenced
this issue
May 26, 2020
# This is the 1st commit message: build(docker): Split builder image out for better caching This is an experimental new approach which creates a separate builder image, that is highly cacheable (and allows caching for yarn install step) to generate the Python wheel. It should save us multiple minutes by leveraging our builds' incremental nature much better. # This is the commit message #2: remove redundant op, trigger subsequent build # This is the commit message #3: add SOURCE_COMMIT env to builder run # This is the commit message #4: oops # This is the commit message #5: add revision and license labels to builder # This is the commit message #6: use kaniko 0.22 as it should fix the weird failures # This is the commit message #7: Simpler and stronger .dockerignore
Closed
JonasBa
added a commit
that referenced
this issue
Dec 8, 2022
JonasBa
added a commit
that referenced
this issue
Dec 8, 2022
This issue was closed.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
I think this dependency should be documented (pip install django-paging, add 'paging' to INSTALLED_APPS).
The text was updated successfully, but these errors were encountered: