token collection: create spec program

[buffalojoec]

Ok, I threw this together to see what we think of it. A reference example of using SPL Token Group to make a Collections program.

[buffalojoec]

Current dependencies on/for this PR:

• master
  • PR token group: add mint to interface #5616
    • PR token-group: create example program #5548
      • PR token 2022: add GroupPointer extension #5621
        • PR token 2022: add InitializeGroup instruction from SPL Token Group interface #5601
          • PR token 2022: add UpdateGroupMaxSize instruction from SPL Token Group interface #5626
            • PR token 2022: add UpdateGroupAuthority instruction from SPL Token Group interface #5627
              • PR token 2022: add GroupMemberPointer extension #5628
                • PR token 2022: add InitializeMember instruction from SPL Token Group interface #5629
      • PR token collection: create spec program #5614 👈
      • PR token-group: create editions example #5549
        • PR token 2022: add support for groups #5593

This comment was auto-generated by Graphite.

[joncinque]

Big fan of the test and the program!

There's a lot of duplicated code between the base example and this one -- it seems like the only difference is the metadata pointer + metadata check, and the ability to have multiple member entries in a group. Anything else?

I'm not sure if there's a good way to resolve that. We could do it through a feature or by exporting everything from the example to be reused here. Or maybe we keep it as is, since it's not too much code. What do you think?

[buffalojoec]

Big fan of the test and the program!

There's a lot of duplicated code between the base example and this one -- it seems like the only difference is the metadata pointer + metadata check, and the ability to have multiple member entries in a group. Anything else?

I'm not sure if there's a good way to resolve that. We could do it through a feature or by exporting everything from the example to be reused here. Or maybe we keep it as is, since it's not too much code. What do you think?

Yeah, I agree there's a lot of similarity. Given that it's really only a processor, I've decided to add a commit for importing the example program's processor functions for process_update_group_max_size and process_update_group_authority (turns out they were already pub!).

I think this is the best we can do with sharing code from the example program. Let me know what you think, and if you think it's not confusing to onlookers.

[joncinque]

Looks great! I do like the reusing of the example program processors, makes them seem more generally useful