Skip to content

Reuse publish state - #1070

Merged
binh-dam-ibigroup merged 5 commits into
devfrom
reuse-publish-state
Oct 5, 2026
Merged

binh-dam-ibigroup merged 5 commits into
devfrom
reuse-publish-state

Conversation

@binh-dam-ibigroup

@binh-dam-ibigroup binh-dam-ibigroup commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

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
  • [na] The description lists all applicable issues this PR seeks to resolve
  • [na] The description lists any configuration setting(s) that differ from the default settings
  • All tests and CI builds passing

Description

MTC config required.

This PR reuses publishState computed by the backend to determine the state of the "Publish to MTC" button.
It also adds a warning about publishing a future feed.

@josh-willis-arcadis josh-willis-arcadis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

much cleaner!

@miles-grant-ibigroup miles-grant-ibigroup left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Clean!

@binh-dam-ibigroup
binh-dam-ibigroup merged commit bc00919 into dev Oct 5, 2026
5 checks passed
@binh-dam-ibigroup
binh-dam-ibigroup deleted the reuse-publish-state branch October 5, 2026 18:34
@binh-dam-ibigroup
binh-dam-ibigroup restored the reuse-publish-state branch October 5, 2026 20:10
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