Skip to content

Implement read RMXP events service#493

Merged
AerunDev merged 3 commits intodevelopfrom
implement-read-rmxp-events-service
May 6, 2025
Merged

Implement read RMXP events service#493
AerunDev merged 3 commits intodevelopfrom
implement-read-rmxp-events-service

Conversation

@Palbolsky
Copy link
Collaborator

@Palbolsky Palbolsky commented Apr 20, 2025

Description

This PR implements a service to read the RMXP events to provide objects that can be used by a converter.

RMXP documentation: https://www.rpg-maker.fr/dl/monos/aide/xp/index.html?page=source%2Frgss%2Frgss.html

The service can be call this:

window.api.readRMXPEvents(
  { projectPath: state.projectPath!, mapId: 5 },
  ({ rmxpEvents }) => console.log(rmxpEvents),
  ({ errorMessage }) => console.error(errorMessage)
)

Note before testing

To test, you can replace the Dashboard.page (in src\views\pages\dashboard) by the following file:
Dashboard.page.zip

You can change the mapId to read an other map.

Tests to perform

  • The service read the RMXP events correctly

Screenshot

Example of an event of the River map:

image

@Palbolsky Palbolsky added the Technical story Issues related to technical work label Apr 20, 2025
@Palbolsky Palbolsky self-assigned this Apr 20, 2025
@Palbolsky Palbolsky moved this from Backlog to In Review in Pokémon Studio Apr 20, 2025
@AerunDev AerunDev added this to the Version 3.0 milestone Apr 30, 2025
@Palbolsky Palbolsky force-pushed the implement-read-rmxp-events-service branch from 20051e8 to 82045d4 Compare May 5, 2025 16:41
@AerunDev AerunDev merged commit a4a6c4a into develop May 6, 2025
5 checks passed
@AerunDev AerunDev deleted the implement-read-rmxp-events-service branch May 6, 2025 21:37
@github-project-automation github-project-automation bot moved this from In Review to Done in Pokémon Studio May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Technical story Issues related to technical work

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants