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

Refresh dashbord name #1323 #1463

Merged

Conversation

atrovato
Copy link
Contributor

@atrovato atrovato commented Mar 12, 2022

Pull Request check-list

To ensure your Pull Request can be accepted as fast as possible, make sure to review and check all of these items:

  • [ ] If your changes affects code, did your write the tests?
  • Are tests passing? (npm test on both front/server)
  • Is the linter passing? (npm run eslint on both front/server)
  • Did you run prettier? (npm run prettier on both front/server)
  • [ ] If you are adding a new features/services, did you run integration comparator? (npm run compare-translations on front)
  • [ ] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community for testing before merging.
  • [ ] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
  • [ ] If you are adding a new features/services which needs explanation, did you modify the user documentation? See the GitHub repo and the website.
  • [ ] Did you add fake requests data for the demo mode (front/src/config/demo.js) so that the demo website is working without a backend? (if needed) See https://demo.gladysassistant.com.

NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.

Description of change

Update dashboard name after edit in dropdown list.

Fixes #1323

@atrovato atrovato requested a review from Pierre-Gilles March 12, 2022 19:02
@atrovato atrovato force-pushed the dashboard-dropdown-update branch from 9ac1b11 to 0872933 Compare March 12, 2022 19:11
@codecov
Copy link

codecov bot commented Mar 12, 2022

Codecov Report

Merging #1463 (0872933) into master (ca30ceb) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1463   +/-   ##
=======================================
  Coverage   96.58%   96.58%           
=======================================
  Files         616      616           
  Lines        8866     8866           
=======================================
  Hits         8563     8563           
  Misses        303      303           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ca30ceb...0872933. Read the comment docs.

Copy link
Contributor

@Pierre-Gilles Pierre-Gilles left a comment

Choose a reason for hiding this comment

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

Works fine on my side! Thanks for the PR !

@Pierre-Gilles Pierre-Gilles merged commit 3800a76 into GladysAssistant:master Mar 18, 2022
@relativeci
Copy link

relativeci bot commented Mar 18, 2022

Job #242: Bundle Size — 6.73MB (~+0.01%).

3800a76 vs 7581b4d

Changed metrics (1/10)
Metric Current Baseline
Cache Invalidation 59.71% 43.25%
Changed assets by type (1/7)
            Current     Baseline
JS 4.82MB (~+0.01%) 4.82MB

View Job #242 report on app.relative-ci.com

@atrovato atrovato deleted the dashboard-dropdown-update branch March 18, 2023 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

After editing a dashboard name, the dropdown list is not updated
2 participants