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/bug fixes #190

Open
wants to merge 18 commits into
base: ui-overhaul
Choose a base branch
from
Open

Fix/bug fixes #190

wants to merge 18 commits into from

Conversation

luketd
Copy link
Collaborator

@luketd luketd commented Oct 20, 2024

Bug Fixes:

  • Bug Report fixed for dark mode
  • removed accept trade / deny trade for completed trades
  • on /latest-pack added hoverable states and clickable states like collection pages
  • Fixed Collection page, trade page, pack page from refreshing. (pass in variables by server side props)
  • Added hoverable states to /packs/[packid] page
  • fixed issue where it shows invalid packID where there shouldnt be
  • in /trades allow user to click out of trade partners dropdown. For keyboard focus, made whole selection highlighted
  • fixed spelling of ' in trades Open [users]'s Cards
  • reroute user to home page if they log out in /packs/last-pack
  • In /packs page, when user is logged out, the buy button turns into a redirect button to login
  • Fixed base pack cover image in /packs
  • Centered the Base Pack Subscription, moved "Base Pack Subscription" to be above the dropdown. Added Toast to inform user that they switched to different subscription
  • Fixed time until midnight (hopefully)
  • Fixed query for Most Recent Packs not showing up first in Latest Pack Opened Section of the main page
  • Changed HOF emoji to 🐐

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment