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

Prep for release. #172

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

weegeekps
Copy link
Contributor

Lots of stuff in here from @javagl, @markSal, and myself. All oriented around getting us ready for public release of the Wordpress hosted version of the project explorer.

@weegeekps weegeekps requested a review from javagl August 1, 2023 21:03
@javagl
Copy link
Contributor

javagl commented Aug 1, 2023

I had a short look at the last commit

  • The IProjectInfo id can probably be removed (I think it is not used any more, and replaced by the key)
  • I'm still wondering whether that additional await has any effect...

I'll try to check and try it out locally soon, but if the build output can go directly into the WordPress site, then this should be fine. (It should probably not be merged anyhow until the other topics are settled)

@javagl
Copy link
Contributor

javagl commented May 15, 2024

@weegeekps Do you remember what's the overall state and role of this?

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