-
Notifications
You must be signed in to change notification settings - Fork 9
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
Not sure, some stashed changes related to button #388
Conversation
…esign # Conflicts: # package-lock.json # package.json
* use real page ids on landing page
The old tags/$tag route is moved to tags/single/$tag. tags/single is used by the questions route and so could be removed if questions route is removed.
Without slash, navigation to question from the tag display page failed. Question id was appended to /tag which isn't a route.
Makes it easier to use component in storybook
This plugin adds fix rule to easily remove unused imports
changes the route for tag pages from tags/{tag name} to tags/{tag id}/{tag name}. Only tag id is needed to navigate to the page.
stampy.ts tests were failing (e.g. https://github.com/StampyAI/stampy-ui/actions/runs/7854687158/job/21435971107) with the following error: ``` app/server-utils/stampy.ts:196:45 - error TS2576: Property 'json' does not exist on type 'Response'. Did you mean to access the static member 'Response.json' instead? 196 json = await (await fetch(url, params)).json() ``` Removed @nodes/types as I think it was due to the types conflict described cloudflare/workerd#1298.
redirect ?state urls (#382 part 1)
@melissasamworth I think you want to make a new branch from Is this the commit with the changes you want to merge? 72855b7 |
feat(Input): add placeholder prop and set for Menu
I'll close this one, as the changes are already merged, and this shouldn't go to master |
No description provided.