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

Fixing Pack Addition Error and Adding Copy Feature #882

Closed
3 tasks done
EngrAJ8888 opened this issue May 1, 2024 · 20 comments
Closed
3 tasks done

Fixing Pack Addition Error and Adding Copy Feature #882

EngrAJ8888 opened this issue May 1, 2024 · 20 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@EngrAJ8888
Copy link
Collaborator

EngrAJ8888 commented May 1, 2024

Describe the bug

  • Implement Copy Option where users can duplicate entire packs created by others, leading them to a new page where they can modify the cloned pack as desired.
  • Users should be able to add items from global. Editing a cloned global item in the pack should not affect the listed item in the global section.
  • Users can't add items when creating a pack. Clicking "Calculate Pack Score" makes all manually added items disappear. Please see video below for reference.
    https://github.com/andrew-bierman/PackRat/assets/122357250/6cca0a80-d791-4bf3-b5e1-bdeea2bdffc0

To Reproduce
Steps to reproduce the behavior:

  1. Visit the URL: https://packrat.pages.dev
  2. Log in to your account with the following credentials:
    Email: zoot3@email.com
    Password: 12345678
  3. Click on the "Create a pack" option, under quick actions.
  4. Begin adding items to your pack
  5. Select 'Calculate Score'

Expected behavior

  1. Users should be able to edit global items they've cloned/added to their pack, and any changes from the global items should not affect the items on the user's pack.
  2. Added items retains unless the user chooses to delete them.
@EngrAJ8888 EngrAJ8888 added bug Something isn't working enhancement New feature or request labels May 1, 2024
@Isthisanmol Isthisanmol self-assigned this May 3, 2024
@Isthisanmol
Copy link
Collaborator

Hi @andrew-bierman, @EngrAJ8888 , I am getting an error on adding item to pack:
""error":{"message":"D1_ERROR: table item has no column named owner_id","code":-32603" I think this is related to database.
Screenshot from 2024-05-03 19-11-04

@EngrAJ8888
Copy link
Collaborator Author

Hi @Isthisanmol, I've checked the live version, and two items are already fixed. The only task left is to add the copy option. Thank you for working on this one! :)

@Isthisanmol
Copy link
Collaborator

Hi @EngrAJ8888, I have checked the second issue on native. It was not working as expected so I fixed it on native. Please check PR: #900 and tell me if any issues or updates.Thanks.

@EngrAJ8888
Copy link
Collaborator Author

EngrAJ8888 commented May 7, 2024

Hi @Isthisanmol , thank you for making the updates!

I'd also like to add this item to the list of needed updates:

  • Fix the pack creation view feature. Currently, after clicking 'Add Pack', users remain on the same page instead of being directed to the Pack Creation page as expected. Consequently, users are required to navigate to the 'Pack' menu to edit newly added packs.

image
Please see photo above - it only stays on this page.

@JewelTee
Copy link
Collaborator

JewelTee commented May 7, 2024

hi @Isthisanmol I want to add this also:

  • The 'edit' option for items in the old packs created is missing, but it's available for the newly created packs.
    please see images below, Thanks!
Screenshot 2024-05-07 at 21 10 26 Screenshot 2024-05-07 at 21 10 03

@Isthisanmol
Copy link
Collaborator

Hi @Isthisanmol , thank you for making the updates!

I'd also like to add this item to the list of needed updates:

* [ ]  Fix the pack creation view feature. Currently, after clicking 'Add Pack', users remain on the same page instead of being directed to the Pack Creation page as expected. Consequently, users are required to navigate to the 'Pack' menu to edit newly added packs.

image Please see photo above - it only stays on this page.

Hi @EngrAJ8888, Please check PR: #906 for the above issue and tell me if any issues or updates.

@Isthisanmol
Copy link
Collaborator

hi @Isthisanmol I want to add this also:

* [ ]  The 'edit' option for items in the old packs created is missing, but it's available for the newly created packs.
  please see images below, Thanks!

Screenshot 2024-05-07 at 21 10 26 Screenshot 2024-05-07 at 21 10 03

Hi @JewelTee, I've checked this. It's not a matter of a new pack v/s an old pack. The owner of the item has the exclusive ability to edit it. When I compare two different items from the same pack, I notice that one only offers the option to delete it, while the other provides both options. This is because I am the owner of the second item, not the first one. Hence, I see both options only for the second item. I'll attach screenshots of both items for reference.
2
1

@Isthisanmol
Copy link
Collaborator

And also @JewelTee Please check PR: #907. The dropdown for edit and delete of items was not working on native. Therefore, I fixed it. Let me know if any issues or updates. Thanks.

@EngrAJ8888
Copy link
Collaborator Author

Hi @Isthisanmol , thank you for making the updates!
I'd also like to add this item to the list of needed updates:

* [ ]  Fix the pack creation view feature. Currently, after clicking 'Add Pack', users remain on the same page instead of being directed to the Pack Creation page as expected. Consequently, users are required to navigate to the 'Pack' menu to edit newly added packs.

image Please see photo above - it only stays on this page.

Hi @EngrAJ8888, Please check PR: #906 for the above issue and tell me if any issues or updates.

Hi @Isthisanmol - The PR has already been merged, and I noticed that the edit and delete options for items in the pack are no longer working. Could you please take a look into it as well? Thanks!

@Isthisanmol
Copy link
Collaborator

Hi @EngrAJ8888, please check PR: #918 and let me know if any issues or updates.

@Isthisanmol
Copy link
Collaborator

Hi @EngrAJ8888, please check PR:#926 for the edit and delete options for items in pack and let me know if any issues or updates.

@EngrAJ8888
Copy link
Collaborator Author

Hi @Isthisanmol, just wanted to let you know that I've added a comment on the PR. Thanks for making the updates!

@Isthisanmol
Copy link
Collaborator

Hi @EngrAJ8888, please check PR: #918 and let me know if any issues or updates.

Any issues in this?

@Isthisanmol
Copy link
Collaborator

Hi @EngrAJ8888, please check PR: #930. I think it resolves the edit options issue. Let me know if edit option is still not working after first edit.

@Isthisanmol
Copy link
Collaborator

Hi @JewelTee @EngrAJ8888 , please check #937. Let me know if the issue persists.

@EngrAJ8888
Copy link
Collaborator Author

Hi @Isthisanmol , I think there's some confusion about the update needed. The "Copy Pack" feature should be available when the owner browses packs created by other users, not on their own packs. The owner can clone packs from the Feed page and can modify the items in the cloned pack as well. Thanks!

@Isthisanmol
Copy link
Collaborator

Hi @Isthisanmol , I think there's some confusion about the update needed. The "Copy Pack" feature should be available when the owner browses packs created by other users, not on their own packs. The owner can clone packs from the Feed page and can modify the items in the cloned pack as well. Thanks!

Should the other options on item and pack also work like delete and save on pack and delete on item.

@EngrAJ8888
Copy link
Collaborator Author

Hi @Isthisanmol , if you are referring to the cloned pack, yes that is correct. Users should have the ability to modify their cloned pack, with options to edit and delete.

@Isthisanmol
Copy link
Collaborator

Hi @Isthisanmol , if you are referring to the cloned pack, yes that is correct. Users should have the ability to modify their cloned pack, with options to edit and delete.

Please Check #937 for the latest commits.

@EngrAJ8888
Copy link
Collaborator Author

Updates have been confirmed and validated. This task can now be marked as closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants