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.

Sign up for GitHub

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

Jump to bottom

Adding products with rest doesn't work when indexes are set to update by schedule #1833

Closed
rklerck opened this issue Sep 10, 2015 · 3 comments
Closed

Adding products with rest doesn't work when indexes are set to update by schedule #1833

rklerck opened this issue Sep 10, 2015 · 3 comments
Assignees
@choukalos
Labels
improvement

Comments

@rklerck
Copy link

rklerck commented Sep 10, 2015

We're adding our products with Rest. There was already one bug with it: #1720

But now there is another one. We set our indexes to update by schedule (config is default). When we add a product with rest, and enable it in backend. It's not visible in the frontend. In the backend everything looks fine. Even when we manually rebuild index (magento indexer:reindex), the product isn't visible.

When we do the same routine with indexes set to on update, it works like expected, except for the bug mentioned earlier.
@rklerck rklerck mentioned this issue Sep 10, 2015
Closed
@choukalos
Copy link

Hi @rklerck when you're doing index update by schedule what happens when the scheduled indexing happens? Does the product then show up on the front end ( that would be expected behavior )?

@choukalos choukalos self-assigned this Oct 2, 2015
@rklerck
Copy link
Author

rklerck commented Oct 19, 2015

It indeed shows up on the front end now after running: "indexer:reindex". Not sure how the whole cron mechanism works though, but that's my problem.

It looks like #1720 isn't a problem anymore, after reindexing manually.

We should test the removing of 85000 products again when using this method, instead of update on save.

@choukalos
Copy link

@rklerck the cron effectively needs to run 2 times to actually update. first time it runs it grabs a # of tasks and converts those tasks to actions that it queues up for the next run. When it runs the 2nd time it processes those actions.

Since you've confirmed we're working correctly I'm going to close this item. I look forward to understanding how the change in indexing strategy impacts #1823 .

Thanks!

magento-team pushed a commit that referenced this issue Dec 12, 2017
@ishakhsuvarov
Merge pull request #1833 from magento-engcom/2.2-develop-prs
5f763a9 
EngCom] Public Pull Requests - 2.2-develop
 - MAGETWO-85520: Remove @escapenotverified from documentation #12639
 - MAGETWO-85317: 12526: Currency change, Bank Transfer but checkout page shows "Your credit card will be charged for" #993
 - MAGETWO-85298: 8011: Strip Tags from attribute. #968
 - MAGETWO-85294: 12535: Product change sku via repository. #984
 - MAGETWO-85274: Update CrontabManager.php #12610
 - MAGETWO-84994: 8862: Can't emptying values by magento 2 api #916
magento-engcom-team added a commit that referenced this issue Sep 23, 2020
@magento-engcom-team
ENGCOM-8223: #1833: Add ability to disable renditions functionality t…
95ba128 
…o stores configuration #30057

 - Merge Pull Request #30057 from joweecaquicla/magento2:ASI-1833-add-ability-to-disable-renditions-functionality-to-stores-configuration
 - Merged commits:
   1. 356ef14
   2. 27f50e4
   3. 1de7ccb
   4. 4e20d29
   5. 746ee90
   6. 24cec36
   7. 2773046
   8. e1bbf88
   9. bb3822a
magento-engcom-team pushed a commit that referenced this issue Sep 23, 2020
@gabrieldagama
ENGCOM-8223: #1833: Add ability to disable renditions functionality t…
7534dbc 
…o stores configuration #30057
magento-engcom-team pushed a commit that referenced this issue Sep 23, 2020
@gabrieldagama
🔃 [Magento Community Engineering] Community Contributions - 2.4-devel…
b2e7753 
…op latest changes

Accepted Community Pull Requests:
 - #30057: #1833: Add ability to disable renditions functionality to stores configuration (by @joweecaquicla)
 - #29794: Fix issue product name in minicart render wrong with special characters (by @mrtuvn)
 - #30034: #1830: Stabilize AdminMediaGalleryCatalogUiVerifyUsedInLinkProductGridTest and AdminMediaGalleryCatalogUiUsedInCategoryFilterTest (by @joweecaquicla)
 - #29814: Fix of Asynchronous opetations status issue (by @nuzil)
 - #29972: Update zip_codes.xml (by @marcostrama)
 - #28818: Improvement PageLayout Config Builder. Added save in cache config files. (by @Knase)
 - #29081: Fix for wildcard section reload being broken by section-config refacor (by @complexthings)
 - #29080: UPS Shipment Tracking: Show "Delivered On" only when package has actually been delivered (by @danbtl)
 - #28164: Prevent side effect on category objects store_id and url_key on save (by @schmengler)
 - #27980: Fix SQL query quoting/casting when type is passed to where function (by @tmotyl)
 - #30095: Added configuration for Stale Issue bot (by @sdzhepa)


Fixed GitHub Issues:
 - #29075: Minicart product name - not covered cases (reported by @Januszpl) has been fixed in #29794 by @mrtuvn in 2.4-develop branch
   Related commits:
     1. 9df9db2

 - #29718: Wrong handling some of asynchronous bulk operations and related mysql queue messages in Magento 2.4.0 (reported by @Neos2007) has been fixed in #29814 by @nuzil in 2.4-develop branch
   Related commits:
     1. 09b63c7
     2. 2450b14
     3. 77d62ef
     4. 4da31ea
     5. a0e65ed
     6. 5c87afe

 - #29984: [Issue] Update zip_codes.xml (reported by @m2-assistant[bot]) has been fixed in #29972 by @marcostrama in 2.4-develop branch
   Related commits:
     1. 1472740
     2. 16b8d6a
     3. 65b5ee4
     4. c336ccc
     5. 2f94846

 - #29546: [Issue] Improvement PageLayout Config Builder. Added save in cache config files. (reported by @m2-assistant[bot]) has been fixed in #28818 by @Knase in 2.4-develop branch
   Related commits:
     1. f2468b5
     2. cc2c5b9
     3. 26b41af
     4. eeecdf7
     5. 2ac6e0c
     6. 276cb1c

 - #28154: [Backward compatibility] Customer section load with one section causes an error (reported by @zhartaunik) has been fixed in #29081 by @complexthings in 2.4-develop branch
   Related commits:
     1. 4f10b5d
     2. 1505a36
     3. ff99146
     4. 8047497
     5. dd862a2
     6. cdfc681
     7. c2e254c
     8. c490073
     9. 176013f
     10. 2922c1c
     11. f7da23e
     12. 3b397e3
     13. 6f9a274
     14. 5de71f6
     15. fd4a2c7
     16. fae1c5c

 - #28428: Customer data not retrieved after executing a custom login controller (reported by @pitbulk) has been fixed in #29081 by @complexthings in 2.4-develop branch
   Related commits:
     1. 4f10b5d
     2. 1505a36
     3. ff99146
     4. 8047497
     5. dd862a2
     6. cdfc681
     7. c2e254c
     8. c490073
     9. 176013f
     10. 2922c1c
     11. f7da23e
     12. 3b397e3
     13. 6f9a274
     14. 5de71f6
     15. fd4a2c7
     16. fae1c5c

 - #29659: [Issue] UPS Shipment Tracking: Show "Delivered On" only when package has actually been delivered (reported by @m2-assistant[bot]) has been fixed in #29080 by @danbtl in 2.4-develop branch
   Related commits:
     1. 5031159
     2. 123a4f0

 - #29585: [Issue] Prevent side effect on category objects store_id and url_key on save (reported by @m2-assistant[bot]) has been fixed in #28164 by @schmengler in 2.4-develop branch
   Related commits:
     1. 7bd5e25
     2. 8bcd2fa
     3. 87c5c60
     4. 31c5963
     5. c051134
     6. 6083a7c

 - #29590: [Issue] Fix SQL query quoting/casting when type is passed to where function (reported by @m2-assistant[bot]) has been fixed in #27980 by @tmotyl in 2.4-develop branch
   Related commits:
     1. 0118d48
     2. b8dd6b3
     3. 6c41578
     4. 4860f31
     5. a103b64
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@choukalos choukalos

Labels
improvement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@choukalos @rklerck @okorshenko