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

OC widget missing in embed [bounty: 8 XTR] #2618

Closed
DreadKnight opened this issue Aug 27, 2024 · 21 comments
Closed

OC widget missing in embed [bounty: 8 XTR] #2618

DreadKnight opened this issue Aug 27, 2024 · 21 comments
Labels
bounty Some amount of our token, XatteR, is offered coding This issue requires some programming funding Generating extra revenue for the project invalid The issue is not from the project or a "won't fix" marketing Spreading the word about the project priority This should get fixed as soon as possible! visuals Various things that easily catch the eye
Milestone

Comments

@DreadKnight
Copy link
Member

When the game is embedded, the Open Collective widget located in dash view under the materialization button is not visible.
Tested in Firefox. There's probably a restriction or problem with showing an iframe within an iframe or something like that.

See https://AncientBeast.com (embedded, no widget) versus https://beta.AncientBeast.com (direct, with OC widget).

Screenshot_20240827_152907

Screenshot_20240827_152936

@DreadKnight DreadKnight added coding This issue requires some programming visuals Various things that easily catch the eye marketing Spreading the word about the project priority This should get fixed as soon as possible! funding Generating extra revenue for the project bounty Some amount of our token, XatteR, is offered labels Aug 27, 2024
@DreadKnight DreadKnight added this to the 0.5 - Chimera milestone Aug 27, 2024
@priyanshunishad4
Copy link

hi, @DreadKnight , i was working to fix this issue , i think i found it , i have attached screenshot , can you tell me is this right? if yes , please assign me this task.

@priyanshunishad4
Copy link

Uploading Screenshot from 2024-09-11 10-31-11.png…

@DreadKnight
Copy link
Member Author

DreadKnight commented Sep 11, 2024

@priyanshunishad4 Heya! I can't see the screenshot just yet... Will assign you to it.

@priyanshunishad4
Copy link

@priyanshunishad4 Heya! Yeah, that's the one. Will assign you to it then.

thank you then , i am exited this one would be my first git hub contribution

@DreadKnight
Copy link
Member Author

@priyanshunishad4 Heya! Yeah, that's the one. Will assign you to it then.

thank you then , i am exited this one would be my first git hub contribution

You answered way too quick, can't see your screenie. Got confused seeing my screenshots as I'm on phone and just woke up 😆

@DreadKnight
Copy link
Member Author

@priyanshunishad4 I guess you didn't finish uploading your screenshot.

@priyanshunishad4
Copy link

i don't know i trying uploading , no problem boss , i will figure it out

@priyanshunishad4
Copy link

Screenshot from 2024-09-11 10-31-11

@DreadKnight
Copy link
Member Author

Screenshot from 2024-09-11 10-31-11

Yeah, that's the one. It's lacking a div around it compared to my screenie though.

@DreadKnight
Copy link
Member Author

@priyanshunishad4 Heya! Any progress with this one? Soft deadline is usually 2 weeks...

@priyanshunishad4 priyanshunishad4 removed their assignment Sep 26, 2024
@priyanshunishad4
Copy link

hey @DreadKnight , i am so sorry for not completing this task , i was busy preparing for interview. i should not take this task. i am unassigning myself.

@codemaster321
Copy link
Contributor

Hi @DreadKnight, I tried testing https://beta.ancientbeast.com/ on firefox, but for me the OC widgets arent visible, but when inspecting them from console. they are visible. Attaching the screenshots

image
image

@DreadKnight
Copy link
Member Author

@codemaster321 I see, might be a CSS related issue with how stuff is revealed on screen and such.

@codemaster321
Copy link
Contributor

codemaster321 commented Oct 15, 2024

@DreadKnight, Yea it is. So could you tell me how do you want this issue to be fixed. I added a overflow-y:auto in the closest div, so here is a small fix. Meanwhile, I will think a solution for oc widget in embed

image

@codemaster321
Copy link
Contributor

codemaster321 commented Oct 16, 2024

Hi @DreadKnight , I tried to troubleshoot so i opened https://play.ancientbeast.com/ and inspected the DOM. I found that there's no OC widget div there.
image.
Furthermore, I think the problem is https://ancientbeast.com/ is pointing to v0.4-stable branch which has no OC widget div there and not to the master branch. So do you want me to push changes in that branch?

@DreadKnight
Copy link
Member Author

Hi @DreadKnight , I tried to troubleshoot so i opened https://play.ancientbeast.com/ and inspected the DOM. I found that there's no OC widget div there.
image.
Furthermore, I think the problem is https://ancientbeast.com/ is pointing to v0.4-stable branch which has no OC widget div there and not to the master branch. So do you want me to push changes in that branch?

Indeed, play subdomain and homepage has v0.4 which is stable and didn't probably include this. Will check patches soon and mark this as invalid and close if so.

@codemaster321
Copy link
Contributor

@DreadKnight, Oh, I wasn't aware of that haha. Now I got it.

@codemaster321
Copy link
Contributor

Also, what should be done regarding the overflow issue for OC widget in v0.5-alpha. Should I open a PR for that?

@DreadKnight
Copy link
Member Author

DreadKnight commented Oct 16, 2024

Also, what should be done regarding the overflow issue for OC widget in v0.5-alpha. Should I open a PR for that?

Yeah, feel free to do so. I'll be able to properly test it out that way.

@DreadKnight
Copy link
Member Author

@codemaster321 Nice catch on these. Looking forward to that PR. Will reward you with XatteR, so make sure you add a wallet address, guide here -> https://github.com/FreezingMoon/AncientBeast/wiki/XatteR#bounties

@DreadKnight DreadKnight added the invalid The issue is not from the project or a "won't fix" label Oct 16, 2024
@codemaster321
Copy link
Contributor

@codemaster321 Nice catch on these. Looking forward to that PR. Will reward you with XatteR, so make sure you add a wallet address, guide here -> https://github.com/FreezingMoon/AncientBeast/wiki/XatteR#bounties

Much thanks. Really appreciate it. I will make a PR by tomorrow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bounty Some amount of our token, XatteR, is offered coding This issue requires some programming funding Generating extra revenue for the project invalid The issue is not from the project or a "won't fix" marketing Spreading the word about the project priority This should get fixed as soon as possible! visuals Various things that easily catch the eye
Projects
None yet
Development

No branches or pull requests

3 participants