Skip to content

Add modal to indicate a successful save in GTFS+ editor#1040

Merged
josh-willis-arcadis merged 1 commit intomtc-deployfrom
gtfs+-save-complete
Aug 28, 2025
Merged

Add modal to indicate a successful save in GTFS+ editor#1040
josh-willis-arcadis merged 1 commit intomtc-deployfrom
gtfs+-save-complete

Conversation

@josh-willis-arcadis
Copy link
Copy Markdown
Contributor

@josh-willis-arcadis josh-willis-arcadis commented Aug 26, 2025

Checklist

  • Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
  • Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
  • The description lists all applicable issues this PR seeks to resolve
  • The description lists any configuration setting(s) that differ from the default settings
  • All tests and CI builds passing
  • The description lists all relevant PRs included in this release (remove this if not merging to master)
  • e2e tests are all passing (remove this if not merging to master)

Description

Adds a modal indicating a successful save and revalidation of GTFS+ data. Also reminds the user to publish the new feed to make the data available.

### Checklist

- [ ] Appropriate branch selected _(all PRs must first be merged to `dev` before they can be merged to `master`)_
- [ ] Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
- [ ] The description lists all applicable issues this PR seeks to resolve
- [ ] The description lists any configuration setting(s) that differ from the default settings
- [ ] All tests and CI builds passing
- [ ] The description lists all relevant PRs included in this release _(remove this if not merging to master)_
- [ ] e2e tests are all passing _(remove this if not merging to master)_

### Description

Please explain the changes you made here and, if not immediately obvious from the code, how they resolve any referenced issues (and PRs if merging to master). Be sure to include all issues being resolved and any special configuration settings that are need for the software to run properly with these changes.
Copy link
Copy Markdown
Contributor

@binh-dam-ibigroup binh-dam-ibigroup left a comment

Choose a reason for hiding this comment

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

Nice one

@josh-willis-arcadis josh-willis-arcadis merged commit 0f5301d into mtc-deploy Aug 28, 2025
2 checks passed
@josh-willis-arcadis josh-willis-arcadis deleted the gtfs+-save-complete branch August 28, 2025 16:53
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.

3 participants