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: added github html url to task request #1793

Merged
merged 6 commits into from
Dec 22, 2023

Conversation

sahsisunny
Copy link
Member

@sahsisunny sahsisunny commented Dec 18, 2023

Issue Ticket Number:-

Description:

In this pull request, I have addressed the issue described in Real-Dev-Squad/website-status#1050. The problem was related to the absence of GitHub issue ticket links for tasks created via task requests.

Is Under Feature Flag

  • Yes
  • No

Database changes

  • Yes
  • No

Breaking changes (If your feature is breaking/missing something please mention pending tickets)

  • Yes
  • No

Is Development Tested?

  • Yes
  • No

Tested in staging?

  • Yes
  • No

Add relevant Screenshot below ( e.g test coverage etc. )

File % Stmts % Branch % Funcs % Lines Uncovered Line #s
website-backend/models 54.74 45.8 52.86 55.11
taskRequests.js 86.31 83.87 86.36 87.35 ...15-351,382,529

Demo

961f4e14-ec34-444c-8c31-d592bdf70789.mp4

Related

PR - Real-Dev-Squad/website-status#1051

@sahsisunny sahsisunny linked an issue Dec 18, 2023 that may be closed by this pull request
4 tasks
Copy link
Contributor

@Ajeyakrishna-k Ajeyakrishna-k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please don't transform the url which is in task requests. Task requests are not tasks, GitHub url should be present on tasks. So change/add the url in tasks document created from task requests.

  • This change will BREAK multiple task requests flow.
  • BREAK the GitHub issue render in task request details page

@sahsisunny sahsisunny changed the title fix: change github issue api to github html url fix: added github html url to task request Dec 20, 2023
satyam73
satyam73 previously approved these changes Dec 21, 2023
Copy link
Member

@satyam73 satyam73 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

constants/urls.ts Outdated Show resolved Hide resolved
test/unit/models/task-requests.test.js Outdated Show resolved Hide resolved
vinit717
vinit717 previously approved these changes Dec 22, 2023
@sahsisunny sahsisunny dismissed Ajeyakrishna-k’s stale review December 22, 2023 15:07

he did not review again after some request

@prakashchoudhary07 prakashchoudhary07 merged commit d70adf7 into develop Dec 22, 2023
3 checks passed
@prakashchoudhary07 prakashchoudhary07 deleted the fix/github-issue-link branch December 22, 2023 16:10
@sahsisunny sahsisunny mentioned this pull request Dec 23, 2023
10 tasks
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.

Issue: Missing Index Check in Firebase Firestore
5 participants