Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

App Refresh Query UI fix on /apps #2632

Merged
merged 1 commit into from
Dec 4, 2021
Merged

App Refresh Query UI fix on /apps #2632

merged 1 commit into from
Dec 4, 2021

Conversation

teallarson
Copy link
Contributor

@teallarson teallarson commented Dec 2, 2021 •
edited

Change description

  • /apps now shows the correct recurringFrequency (before it was stuck at "every minute")
  • page shows updates after hitting "run"

Screen Shot 2021-12-02 at 2 12 55 PM

Checklists

Development

  • Application changes have been tested appropriately

Impact

  • Code follows company security practices and guidelines
  • Security impact of change has been considered
  • Performance impact of change has been considered
  • Possible migration needs considered (model migrations, config migrations, etc.)

Please explain any security, performance, migration, or other impacts if relevant:

Code review

  • Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached where applicable.
  • Relevant tags have been added to the PR (bug, enhancement, internal, etc.)

@teallarson teallarson added the bug Something isn't working label Dec 2, 2021
@teallarson teallarson changed the title fix recurringFrequency, update when run App Refresh Query UI fix on /apps Dec 3, 2021
@teallarson teallarson merged commit ce69a92 into main Dec 4, 2021
@teallarson teallarson deleted the 180382277-apps-page-updates branch December 4, 2021 02:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@evantahler evantahler evantahler approved these changes

Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@teallarson @evantahler