Skip to content
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

Changes to make screens more responsive + incident status toggler change #1237

Merged
merged 40 commits into from
Feb 7, 2023

Conversation

teodosii
Copy link
Member

@teodosii teodosii commented Jan 27, 2023

What this PR does

  • Changes to make a few screens be more responsive
  • Removed incident actions and replaced incident status with a toggler
  • Renamed IncidentStatus.new to IncidentStatus.Firing
  • Removed old schedules code (unused)

Which issue(s) this PR fixes

#1000

Checklist

  • CHANGELOG.md updated

@teodosii teodosii requested a review from a team January 27, 2023 11:21
@teodosii teodosii added the pr:no public docs Added to a PR that does not require public documentation updates label Jan 30, 2023
@teodosii teodosii requested a review from a team January 30, 2023 09:21
@teodosii teodosii changed the title WIP: Changes to make screens more responsive + incident status toggler change Changes to make screens more responsive + incident status toggler change Jan 30, 2023
@joeyorlando joeyorlando removed the request for review from a team January 30, 2023 11:37
const handleDateChange = useCallback(
(date: Date) => {
onChange({ selectedDate: moment(date).format('YYYY-MM-DD') });
const SchedulesFilters = (props: SchedulesFiltersProps) => {
Copy link
Member Author

@teodosii teodosii Feb 2, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, I haven't touched these parts. Don't know why these changes show up in here

These show up as changes because

  1. Old files of Schedules_NEW were moved into Schedules
  2. Did minor changes to rendering

@maskin25
Copy link
Contributor

maskin25 commented Feb 2, 2023

grafana-plugin/src/containers/AlertRules/AlertRules.tsx conflict detected

@teodosii teodosii requested a review from maskin25 February 2, 2023 11:15
@teodosii teodosii merged commit 81b5741 into dev Feb 7, 2023
@teodosii teodosii deleted the rares/responsive-rc-tables branch February 7, 2023 11:29
brojd pushed a commit that referenced this pull request Sep 18, 2024
…nge (#1237)

# What this PR does

- Changes to make a few screens be more responsive
- Removed incident actions and replaced incident status with a toggler
- Renamed `IncidentStatus.new` to `IncidentStatus.Firing`
- Removed old schedules code (unused)

## Which issue(s) this PR fixes

#1000 

## Checklist

- [x] `CHANGELOG.md` updated

---------

Co-authored-by: Joey Orlando <joey.orlando@grafana.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:no public docs Added to a PR that does not require public documentation updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants