Skip to content

Issue with the package Boron B,5 #524

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

Open
samsil2012 opened this issue Apr 13, 2025 · 11 comments
Open

Issue with the package Boron B,5 #524

samsil2012 opened this issue Apr 13, 2025 · 11 comments
Assignees
Labels
bug Something isn't working

Comments

@samsil2012
Copy link

Describe the bug
Post install, I always get this error

Image
@samsil2012 samsil2012 added the bug Something isn't working label Apr 13, 2025
@samsil2012
Copy link
Author

I installed in a couple of sandboxes, and all the issues seems to be the same.

@VinceFINET
Copy link
Collaborator

VinceFINET commented Apr 13, 2025 via email

@samsil2012
Copy link
Author

@VinceFINET Can you for the time being provide an old stable version of the package(if possible)? It would be super! I really use this intensively and is a baseline for every conversation with the customers (Greenfield vs Brownfield). Thank you for the great tool honestly!

@VinceFINET
Copy link
Collaborator

can you give me your org id please ?

@VinceFINET
Copy link
Collaborator

also can you check if your user is assigned to the Org Check permission set?

@VinceFINET
Copy link
Collaborator

VinceFINET commented Apr 14, 2025 via email

@samsil2012
Copy link
Author

@VinceFINET Org ID: 00D9V00000EBMtV & Org ID: 00DKJ000000LFvp. I already add the permission set to my user and tried synching. Let me try in a an anonymous window

@samsil2012
Copy link
Author

Anonymous Window doesn't work.

@samsil2012
Copy link
Author

It works now! I cleared all the cookies

@VinceFINET
Copy link
Collaborator

Need to make that cache managment a little safer and stronger in the code.
Pushing this ticket to Carbon version as a reminder to myself.

@VinceFINET VinceFINET added this to the Version Carbon [C, 6] milestone Apr 15, 2025
@VinceFINET
Copy link
Collaborator

So basically i need the set method in the implementation of the cache manager to be safer.

Typically here the cache of the browser was in a form that the old version was supporting but not the new one.
I though I covered that use case, but not entirely.
In this scenario, the set method is called with "value" as a Map, but somehow the entries are not passing the current code, triggering the error @samsil2012 is showing in this ticket.

I want to make sure whatever happens in the code (aka an exception) the cache is reset making sure we restart from a fresh data that we support. instead of showing this error.

I'll add some testing data to reproduce this error.

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

No branches or pull requests

2 participants