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

🐞 Space: Fix broken forms and links when on a BYODomain #1083

Closed
wants to merge 2 commits into from

Conversation

zspencer
Copy link
Member

We probably want to figure out a better strategy for how we test the BYO domains feature; perhaps there's a way to do it with the end-to-end tests so that they run against a BYO-domained Space and a non-BYO domained space in different github actions? I don't know 🤷

#74

We probably want to figure out a better strategy for how we test the BYO
domains feature; perhaps there's a way to do it with the end-to-end
tests so that they run against a BYO-domained Space and a non-BYO
domained space in different github actions? I don't know 🤷

#74
@zspencer zspencer added the 🐞 bug Something isn't working label Feb 7, 2023
@zspencer zspencer changed the title BYO Domains: (WIP) Identify a few places where things are shady Space: Fix broken forms and links when on a BYODomain Feb 7, 2023
@zspencer zspencer changed the title Space: Fix broken forms and links when on a BYODomain 🐞 Space: Fix broken forms and links when on a BYODomain Mar 3, 2023
zspencer added a commit that referenced this pull request Mar 3, 2023
- Extracted from htps://github.com//pull/1083

This is mostly for consistency and to start to close out a piece of work
I started a while back but never finished.
zspencer added a commit that referenced this pull request Mar 3, 2023
- Extracted from htps://github.com//pull/1083

This is mostly for consistency and to start to close out a piece of work
I started a while back but never finished.
zspencer added a commit that referenced this pull request Mar 3, 2023
zspencer added a commit that referenced this pull request Mar 4, 2023
zspencer added a commit that referenced this pull request Mar 4, 2023
zspencer added a commit that referenced this pull request Mar 4, 2023
🧹 `Space`: Use `Space#location` for consistency

- #74
- Extracted from: #1083
zspencer added a commit that referenced this pull request Mar 4, 2023
zspencer added a commit that referenced this pull request Mar 4, 2023
🧹 `Furniture`: use `location` rather when building new Furniture

- Extracted from: #1083
- #74
- #709
zspencer added a commit that referenced this pull request Mar 4, 2023
🧹 `Utility`: Use `#location` in forms

- Extracted from htps://github.com//pull/1083

This is mostly for consistency and to start to close out a piece of work
I started a while back but never finished.
@zspencer zspencer closed this Mar 4, 2023
@zspencer zspencer deleted the byo-domains/use-location-in-more-places branch March 4, 2023 01:06
@zspencer
Copy link
Member Author

zspencer commented Mar 4, 2023

And with that, I think all the changes are gone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant