Jieiku / abridge

Fast & Lightweight Zola Theme
https://abridge.pages.dev/
MIT License
166 stars 44 forks source link

Netlify deploy uses abridge.netlify.app domain instead of netlify subdomain #159

Closed Brandon689 closed 9 months ago

Brandon689 commented 9 months ago

Im not sure why, changing base url in config doesnt affect it, then of course the website loads with no css / js because of cors. From what I understand $DEPLOY_PRIME_URL should automatically resolve to the correct domain.

Jieiku commented 9 months ago

in config.toml do you have pwa = true or pwa = false ?

If you have true, then did you run:

npm install
npm run abridge

prior to deployment?

I am not certain this is the cause of your problem, it is just a guess without looking at your sites config files and the feedback from netlify deploy.

If you do resolve this please let me know what fixed it, so that I can update the documentation to help future users.

Jieiku commented 9 months ago

another suggestion is to try with pwa = false initially and redeploy.

Then clear your browser cache or hold down control and press F5 to force a reload that bypasses all cache. It is important to clear the cache, because the pwa may serve from the cache files otherwise.

Then see if the page loads properly.

Brandon689 commented 9 months ago

Despite changing pwa to false, have run npm install npm run abridge, deleting my site, etc, it still remains the abridge domain. Here is my repo, ignore the fact I just changed it now, cause I had like 2 other repos previously awhile ago.

https://github.com/Brandon689/blognetlify

Jieiku commented 9 months ago

Thanks for the repo, testing it out now.

Jieiku commented 9 months ago

Just wanted to let you know I am seeing the same thing https://zola-themes-demos.netlify.app/

I am eating dinner and it is close to bed time, but will figure this out once I examine it closer.

I am sure it is something simple that I am overlooking.

Brandon689 commented 9 months ago

Thank you, no need to rush, I'm doing other work for now.

Jieiku commented 9 months ago

You have to set both in config.toml:

base_url = "https://example.netlify.app"
online_url = "https://example.netlify.app"

online_url exists to facilitate generating an offline site then restoring the base_url for online site. I can see this will be problematic for new users, even I forgot about adding this functionality.

I may comment the field out by default for now, but if you set both of those fields you should be good to go.

Brandon689 commented 9 months ago

No, here is the problem I think:

  1. rsync theme config and site config as instructed
  2. git submodule the theme
  3. i change my local git repo to my domain
  4. the submodule constantly changes my github repo back to abridge.netlify.com

The config file went back to default today after I changed it without me doing anything. Because I think awhile ago I already knew about the online_url and changed that too

Jieiku commented 9 months ago

seems to be working for me

repo: https://github.com/Brandon689/blognetlify/compare/main...Jieiku:blognetlify:main

site: https://zola-themes-demos.netlify.app/

Brandon689 commented 9 months ago

I've got it to work now, well not quite because its now using my domain which take a bit of time for DNS, but I think the problem I mentioned is real. So I would add some docs about deploying to netlify and from github. Its actually confusing because when you make a netlify site you dont have the domain they give you, it doesnt exist yet. So you have to re-deploy.

Also some comment to check your remote origin that the config.toml of your site is correct, not just your local. Finally I would move online_url to be below base_url in config maybe.

Jieiku commented 9 months ago

When I originally cloned your repo I was having the same issue you pointed out, at that point in time online_url was not set and was still abridge.netlify.app which is why the base_url kept changing to that.

I have since made some changes to online_url: https://github.com/Jieiku/abridge/commit/ae68172b9b6a2bdbe620fd36589a6151cee5df28

I attempted to retest this, and can no longer reproduce the issue. I believe the root cause was online_url and it may have also been made worse if the pwa was active.

Jieiku commented 9 months ago

Thanks again for this report, my hope is that abridge will now be easier to use for all.

The online_url is only necessary for those that test offline = true

Most people will not be using offline = true so having online_url commented out by default should make things simpler.