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

Fix: publish button delay #752

Merged
merged 3 commits into from
May 5, 2023
Merged

Conversation

alexanderleegs
Copy link
Contributor

This PR adds an additional guard when publishing changes to check if the review request has been approved. To be reviewed in conjunction with PR #1263 on the isomer cms frontend repo.

Copy link
Contributor

@kishore03109 kishore03109 left a comment

Choose a reason for hiding this comment

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

hey, would it be ok if we shifted this check before calling mergeReviewRequest?

Idea is that we can directly return this as a 404 rather than throwing it and letting the error bubble up, wdyt?
Screenshot 2023-05-05 at 9 02 45 AM

@alexanderleegs
Copy link
Contributor Author

63efe6e

Sure, changed in 63efe6e

@alexanderleegs alexanderleegs merged commit 6ad8e8d into develop May 5, 2023
@mergify mergify bot deleted the fix/publish-button-delay branch May 5, 2023 07:23
This was referenced May 8, 2023
alexanderleegs added a commit that referenced this pull request May 9, 2023
* fix: close pull request (#751)

* Fix: publish button delay (#752)

* Fix: publish button delay

* Fix: tests

* Fix: move review request status check into router

* feat: return sites from db for email login (#754)

* feat: return sites from db for email login

* fix: tests

* fix: use lodash function for type guard

* fix(githhub service): get call to github to prevent race condition (#756)

* fix(githhub service): get call to github to prevent race condition

* fix(github service): throw error if directory name not defined

* style(github service): add comments for clarity

* fix: tests

---------

Co-authored-by: Alexander Lee <[email protected]>

* Fix: handle updating of files in root directory (#761)

* fix: create .keep first when rename subfolder (#762)

* 0.26.0

---------

Co-authored-by: seaerchin <[email protected]>
Co-authored-by: Alexander Lee <[email protected]>
Co-authored-by: Harish <[email protected]>
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.

2 participants