Skip to content

Adds query params to extension requests page url#666

Merged
iamitprakash merged 14 commits intodevelopfrom
feat/query-params-extension-card
Jan 6, 2024
Merged

Adds query params to extension requests page url#666
iamitprakash merged 14 commits intodevelopfrom
feat/query-params-extension-card

Conversation

@Ajeyakrishna-k
Copy link
Contributor

@Ajeyakrishna-k Ajeyakrishna-k commented Dec 31, 2023

Date: 31 December 2023

Developer Name: @Ajeyakrishna-k


Issue Ticket Number

Description

  • Adds colours based on deadline to extension request dates
  • Adds query params to url to save the page state
  • Loads the page with respect to filters and sort specified in query params

Documentation Updated?

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Tested in Staging?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Screenshots

Screen.Recording.2023-12-31.at.3.57.50.PM.mov

Test Coverage

@Ajeyakrishna-k Ajeyakrishna-k changed the title feat: adds query params to extension requests page url Adds query params to extension requests page url Dec 31, 2023
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 1, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: eb1aea1
Status: ✅  Deploy successful!
Preview URL: https://ec4708aa.dashboard-rds.pages.dev
Branch Preview URL: https://feat-query-params-extension.dashboard-rds.pages.dev

View logs

ajoykumardas12
ajoykumardas12 previously approved these changes Jan 2, 2024
satyam73
satyam73 previously approved these changes Jan 5, 2024
Base automatically changed from fix/ext-cards-changes to develop January 5, 2024 15:48
@iamitprakash iamitprakash dismissed stale reviews from satyam73 and ajoykumardas12 January 5, 2024 15:48

The base branch was changed.

@iamitprakash iamitprakash merged commit 8fa2e86 into develop Jan 6, 2024
@iamitprakash iamitprakash deleted the feat/query-params-extension-card branch January 6, 2024 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants